-
Notifications
You must be signed in to change notification settings - Fork 537
-
Notifications
You must be signed in to change notification settings - Fork 537
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 integration test for testing WHERE clause in QWATCH. #188
Comments
Hi @JyotinderSingh I would like to pick this. |
Assigned |
Issue is unassigned now. Feel free to pick this up. |
Hi @JyotinderSingh I would like to pick this. |
Assigned. |
@JyotinderSingh is there any way to overcome comparison issue between string and integer. I am setting up integer in SDK test case but while comparison in where condition, it is giving me error that can't compare string and integer. |
The issue was fixed recently, please pull the latest master |
@deep-adeshraa Thanks a lot for picking this up. Do you have any updates on it? If you have any blockers please discuss this over the discord. |
Hi @AshwinKul28 I have started working on it but having some difficulties in understanding code. I will try to push asap |
Hi @JyotinderSingh @AshwinKul28 - I am working on benchmarking the Qwatch, however I am not sure how to benchmark the code. I am checking |
A good strategy might be to check performance of watch queries with high number of incoming updates to the query keyset. The watch query will be looking at a rapidly changing keyset, we should see how long it takes for each new query result to get delivered. Additionally, another benchmark can be to validate query performance with several queries running in parallel. In such a case you could benchmark both query and SET performance (because each SET operation will compete against the query Executor for lock on the dataset in the current implementation) |
Hello @deep-adeshraa, There has been no activity on this issue for the past 5 days. We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work. Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out. Thanks again. |
Hello @arpitbbhayani - I have created PR for integration test cases. However for Benchmark JyotiderSingh asked me to wait until some fixes are deployed. The reason was QWATCH command was failing if I fire large amount of concurrent requests. |
WHERE clause support was added to DSQL in #187. However, we don't have any integration test cases validating its behavior.
Requirements
The text was updated successfully, but these errors were encountered: