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

Make deprecation around QueryOptionsHandler and QueryOptionsBuilder more consistent #176

Closed
nikos opened this issue Nov 4, 2014 · 1 comment

Comments

@nikos
Copy link
Contributor

nikos commented Nov 4, 2014

We have a search options (named "all") on our project (using ML 7.0.4) where all constraints are defined. However in certain searches we only want to retrieve facet values for certain constraints.

One way to do this is to define the constraint at runtime using the QueryOptionsHandle – however this has been deprecated.

Using the Java Client API (2.04) to solve this, we were using QueryOptionsManager to read options "all" first. Then we wanted to retrieve facet definition (QueryOptionsHandle#getConstraint()).

http://docs.marklogic.com/javadoc/client/com/marklogic/client/io/QueryOptionsHandle.html

Unfortunately it turns out that the returned handle (QueryOptions.QueryConstraint) is deprecated, although getConstraint is not (should it be deprecated also?).

@sammefford sammefford self-assigned this Nov 6, 2014
@sammefford sammefford added this to the 7.0-5 milestone Nov 6, 2014
@sammefford sammefford modified the milestones: 7.0-5, 7.0-6 Feb 12, 2015
@grechaw grechaw removed this from the 7.0-6 milestone Feb 25, 2015
@sammefford sammefford added this to the java-client-api-2.0.6 milestone Oct 22, 2015
sammefford added a commit that referenced this issue Nov 6, 2015
sammefford added a commit that referenced this issue Nov 6, 2015
…ons which is already deprecated

(cherry picked from commit 338f4e3)
@sammefford
Copy link
Contributor

Thanks, Niko, for reporting this. We are deprecating QueryOptionsHandle as it depends on QueryOptions which is deprecated. In place of QueryOptionsHandle please use your favorite XML or JSON handle to read & write query options.

The fix is on both develop and 2.0-develop branches.

@sammefford sammefford added ship and removed fix labels Nov 6, 2015
@sammefford sammefford removed their assignment Nov 6, 2015
sammefford added a commit that referenced this issue Mar 10, 2016
…ons which is already deprecated

(cherry picked from commit 338f4e3)
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