Skip to content

regenerate Optic expression functions #1317

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

Closed
ehennum opened this issue Sep 7, 2021 · 4 comments
Closed

regenerate Optic expression functions #1317

ehennum opened this issue Sep 7, 2021 · 4 comments

Comments

@ehennum
Copy link
Contributor

ehennum commented Sep 7, 2021

pick up the latest changes

@ehennum ehennum added this to the java-client-api-NEXT milestone Sep 7, 2021
@ehennum ehennum self-assigned this Sep 7, 2021
ehennum pushed a commit that referenced this issue Sep 7, 2021
@ehennum
Copy link
Contributor Author

ehennum commented Sep 7, 2021

Added sql.bucket() and cts.columnRangeQuery()
Fixed the data types and documentation on some existing methods.

@ehennum ehennum assigned georgeajit and unassigned ehennum Sep 7, 2021
@ehennum ehennum added test and removed new labels Sep 7, 2021
@ehennum
Copy link
Contributor Author

ehennum commented Sep 7, 2021

Should cause no regressions.

@georgeajit
Copy link

Verified Jenkins nightly build on Oct 10, 2021 with 10.0 MarkLogic Server build. Able to see com.marklogic.client.test.PlanGeneratedTest.testSqlBucket2Exec, but not columnRangeQuery.

Maybe I am missing something here. Could you please clarify?

@ehennum
Copy link
Contributor Author

ehennum commented Oct 4, 2021

Sorry for the delay: The CtsExpr interface has the columnRangeQuery() method:

https://github.com/marklogic/java-client-api/blob/develop/marklogic-client-api/src/main/java/com/marklogic/client/expression/CtsExpr.java#L303

Is there an issue with using it?

@ehennum ehennum assigned georgeajit and unassigned ehennum Oct 4, 2021
@ehennum ehennum added test and removed verify labels Oct 4, 2021
@georgeajit georgeajit added ship and removed test labels Oct 7, 2021
@llinggit llinggit closed this as completed Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants