-
Notifications
You must be signed in to change notification settings - Fork 160
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
Add Quickwit support #95
Comments
@zamazan4ik Oh that would be awesome. Unfortunately Quickwit does not handle SQL at the moment. |
I think it would be ok, since here authors mention the possibility to improve performance via rewriting queries into the native DSL. |
I had a look at the bench. Maybe we cannot perform all of the queries yet, but it seems like the benchmark is pretty flexible and will run the queries we can run. The set of queries is not really friendly to search engines, but it should still be very instructive for us. |
It's ok to add a DBMS without SQL support. |
Looking forward to seeing Quickwit added! |
Would be nice to see Quickwit comparison as well since at least ElasticSearch is supported here right now.
Pinging @fulmicoton - probably you could be interested in it at least from the perspective "where Quickwit performance could be improved compared to other engines".
The text was updated successfully, but these errors were encountered: