-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[docs] document minTimeUUID & maxTimeUUID in YCQL #12351
Conversation
✅ Deploy Preview for infallible-bardeen-164bc9 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
A few comments
@@ -117,6 +117,98 @@ ycqlsh:example> SELECT todate(ts) FROM test_todate; | |||
2018-10-09 | |||
``` | |||
|
|||
## minTimeUUID(<timestamp>) | |||
|
|||
This function generates corresponding (`TIMEUUID`) with minimum node/clock component so that it includes all regular |
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.
Is it a node or clock or both?
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.
Was taken from #801 (comment). I believe node/clock is correct?
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.
@lizayugabyte please recheck
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.
@ddorian In tech docs, it needs to be either one or the other or both. The use of slash (1) creates ambiguity and (2) accessibility readers can't interpret slash between two words.
|
||
## maxTimeUUID(<timestamp>) | ||
|
||
This function generates corresponding (`TIMEUUID`) with maximum node/clock component so that it includes all regular |
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 same question about node/clock
+ fix a query that was using the wrong table
@lizayugabyte please recheck |
fixes #804