Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The qlever start command should stop in case of error #143

Open
ludovicm67 opened this issue Feb 24, 2025 · 0 comments
Open

The qlever start command should stop in case of error #143

ludovicm67 opened this issue Feb 24, 2025 · 0 comments

Comments

@ludovicm67
Copy link
Contributor

Using version 0.5.18 (and it was already the case on the previous ones), it seems that the qlever start command is not giving the prompt back to the user in case of an error.

A simple way to reproduce this, is to try to start a QLever instance without having the index generated.

Running qlever start displays the error, but doesn't stop the execution.
That means, it's complicated for shell script to know that an issue occurred, as the prompt is not given back to the script, and no relevant exit code is returned.

It's possible to listen the logs for "ERROR" or so, and then force the kill of the process, but the issue with this is that it might also kill the process in case a user tries to run an invalid request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant