-
Notifications
You must be signed in to change notification settings - Fork 88
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
Added FILTERS.timestamp().exact()
for better access & read ability.
#92
Conversation
2512606
to
8732523
Compare
Codecov Report
@@ Coverage Diff @@
## master #92 +/- ##
============================================
+ Coverage 81.33% 81.33% +<.01%
Complexity 937 937
============================================
Files 95 95
Lines 5849 5850 +1
Branches 325 325
============================================
+ Hits 4757 4758 +1
Misses 916 916
Partials 176 176
Continue to review full report at Codecov.
|
…lity This change introduces `FILTERS.timestamp().exact()`, which enable user to directly provide exact timestamp to query Bigtable.
8732523
to
ae96870
Compare
.setTimestampRangeFilter( | ||
TimestampRange.newBuilder() | ||
.setStartTimestampMicros(20_000L) | ||
.setEndTimestampMicros(20_000L + 1)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why + 1
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TimestampRange#endTimestampMicros
denotes upper bound for timestamp range. If we pass the same value then no rows will return.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The end range cap is exclusive, so for a single number, the range is [x, x+1)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Fixes #65