-
Notifications
You must be signed in to change notification settings - Fork 192
Add override of toN1qlRemoveString() to StringQuery. #1135
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 override of toN1qlRemoveString() to StringQuery. #1135
Conversation
94dd216
to
f1fb77a
Compare
@@ -323,18 +323,24 @@ public String toN1qlRemoveString(ReactiveCouchbaseTemplate template, String coll | |||
* @return QueryOptions | |||
*/ | |||
public QueryOptions buildQueryOptions(QueryScanConsistency scanConsistency) { | |||
final QueryOptions options = QueryOptions.queryOptions(); | |||
QueryOptions options = QueryOptions.queryOptions(); | |||
if (options == null) { // add/override what we got from PseudoArgs |
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.
that line does not make sense, since the line above you always set it to non-null?
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.
Right. The code is copied from the Collections branch to reduce the future merge. And in that code 'options' is an argument and needs to be checked.
@@ -73,4 +73,16 @@ public String toN1qlSelectString(ReactiveCouchbaseTemplate template, String coll | |||
appendSkipAndLimit(statement); | |||
return statement.toString(); | |||
} | |||
|
|||
@Override |
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.
move below block comment?
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.
Yes. Will fix.
f1fb77a
to
e9e8411
Compare
Closes #1131. Original pull request: #1135. Co-authored-by: mikereiche <[email protected]>
Closes #1131.