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

Optimize for 'Less' and 'Greater' query on Timestamp #3344

Merged
merged 8 commits into from
Aug 15, 2019

Conversation

jedelbo
Copy link
Contributor

@jedelbo jedelbo commented Aug 12, 2019

No description provided.

@realm-probot
Copy link

realm-probot bot commented Aug 12, 2019

Hey - looks like you forgot to add a T:* label - could you please add one?

@jedelbo jedelbo requested a review from ironage August 12, 2019 18:36
@jedelbo jedelbo force-pushed the optimize-timestamp-query branch from 7f5a6cd to fc722cf Compare August 13, 2019 06:35
@jedelbo jedelbo force-pushed the optimize-timestamp-query branch 2 times, most recently from f152d56 to 2fc0334 Compare August 15, 2019 11:59
We take the position that you cannot query a timestamp column
using <, >, <= and >= against a null value.
@jedelbo jedelbo force-pushed the optimize-timestamp-query branch from 2fc0334 to 2e70935 Compare August 15, 2019 12:17
@jedelbo jedelbo requested review from finnschiermer and removed request for ironage August 15, 2019 12:25
@jedelbo jedelbo merged commit 6bd8de6 into master Aug 15, 2019
@realm-probot
Copy link

realm-probot bot commented Aug 15, 2019

Hey - looks like you forgot to add a T:* label - could you please add one?

@jedelbo jedelbo deleted the optimize-timestamp-query branch August 15, 2019 14:49
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants