Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-2627

Need a new parameter for FTS parameter (score).

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.4
    • 3.1.0
    • library
    • None
    • 1
    • SDK48: FTS Score/Incl, Docs.
    • Critical

    Description

      with Couchbase 6.6, we introduced an option to avoid scoring and was deed useful for searching and flex-index from the query service.

      We've seen that this optimization has additional benefits for direct FTS querying. We need this exposed from the SDKs as well. 

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            keshav Keshav Murthy created issue -
            jmorris Jeff Morris made changes -
            Field Original Value New Value
            Affects Version/s 3.0.4 [ 17014 ]
            jmorris Jeff Morris made changes -
            Status New [ 10003 ] Open [ 1 ]

            Keshav Murthy can you also attach the design spec that would tell how and where the parameter need to be passed ?

            arun.vijayraghavan Arun Vijayraghavan added a comment - Keshav Murthy  can you also attach the design spec that would tell how and where the parameter need to be passed ?
            keshav Keshav Murthy added a comment - - edited

            Arun Vijayraghavan : See the doc examples for Limit and Timeout below.  score option should be at the same level.

            ClusterHelper.Initialize(new ClientConfiguration());
            var bucket = ClusterHelper.GetBucket("travel-sample");
            var results = bucket.Query(new SearchQuery
            {
                Index = "idx_travel_content",
                Query = new MatchQuery("inn"),
                SearchParams = new SearchParams().Limit(10).Timeout(TimeSpan.FromMilliseconds(10000))
            });foreach (var result in results)
            {
                Console.WriteLine(result);
            }

             

             

            keshav Keshav Murthy added a comment - - edited Arun Vijayraghavan  : See the doc examples for Limit and Timeout below.  score option should be at the same level. ClusterHelper.Initialize( new ClientConfiguration()); var bucket = ClusterHelper.GetBucket( "travel-sample" ); var results = bucket.Query( new SearchQuery { Index = "idx_travel_content" , Query = new MatchQuery( "inn" ), SearchParams = new SearchParams().Limit( 10 ).Timeout(TimeSpan.FromMilliseconds( 10000 )) });foreach (var result in results) { Console.WriteLine(result); }    

            Thanks Keshav, so probably a boolean UseScoring(true / false) ?

            arun.vijayraghavan Arun Vijayraghavan added a comment - Thanks Keshav, so probably a boolean UseScoring(true / false) ?

            Abhinav Dangeti: Can you please share the actual values... we use "score":"none" for turning this off.  What's the value to run this on? 

            keshav Keshav Murthy added a comment - Abhinav Dangeti : Can you please share the actual values... we use "score":"none" for turning this off.  What's the value to run this on? 
            jmorris Jeff Morris made changes -
            Fix Version/s 3.0.5 [ 17033 ]
            jmorris Jeff Morris made changes -
            Fix Version/s 2.7.21 [ 17044 ]
            jmorris Jeff Morris added a comment - - edited

            .NET, Java and other SDK's have a "raw" options parameter that can be used to pass `{"score":"none"} with the query:

            var options = new SearchOptions().Raw("score", "none"); 

            This should satisfy the requirements of this ticket for now. In 3.1 we can add a strongly typed version to SearchOptions class.

            NOTE: in sdk2 there is no "raw" parameter support for SearchParams - this ticket adds it: NCBC-2629

            -Jeff

            jmorris Jeff Morris added a comment - - edited .NET, Java and other SDK's have a "raw" options parameter that can be used to pass `{"score":"none"} with the query: var options = new SearchOptions().Raw( "score" , "none" ); This should satisfy the requirements of this ticket for now. In 3.1 we can add a strongly typed version to SearchOptions class. NOTE: in sdk2 there is no "raw" parameter support for SearchParams - this ticket adds it: NCBC-2629 -Jeff
            jmorris Jeff Morris made changes -
            Fix Version/s 3.1.0 [ 16741 ]
            Fix Version/s 3.0.5 [ 17033 ]
            jmorris Jeff Morris made changes -
            Fix Version/s 2.7.21 [ 17044 ]
            jmorris Jeff Morris made changes -
            Affects Version/s 2.7.20 [ 17034 ]

            This is good. Thank you, Jeff Morris.

            Girish Benakappa: Can you please test this with .NET api and ensure you're seeing the performance difference? Thanks.

            keshav Keshav Murthy added a comment - This is good. Thank you, Jeff Morris . Girish Benakappa : Can you please test this with .NET api and ensure you're seeing the performance difference? Thanks.

            Keshav Murthy I am passing on the validation for the .NET API to Arunkumar Senthilnathan since he owns the SDK QE.

            mihir.kamdar Mihir Kamdar (Inactive) added a comment - Keshav Murthy I am passing on the validation for the .NET API to Arunkumar Senthilnathan since he owns the SDK QE.
            arun.vijayraghavan Arun Vijayraghavan made changes -
            Labels sdk-6.6 sdkapi-3.1
            arun.vijayraghavan Arun Vijayraghavan made changes -
            Issue Type Bug [ 1 ] Improvement [ 4 ]
            arun.vijayraghavan Arun Vijayraghavan made changes -
            Issue Type Improvement [ 4 ] Bug [ 1 ]
            arun.vijayraghavan Arun Vijayraghavan made changes -
            Labels sdk-6.6 sdkapi-3.1
            jmorris Jeff Morris made changes -
            Priority Critical [ 2 ] Major [ 3 ]
            jmorris Jeff Morris made changes -
            Issue Type Bug [ 1 ] Improvement [ 4 ]
            jmorris Jeff Morris made changes -
            Link This issue duplicates NCBC-2631 [ NCBC-2631 ]

            Arunkumar Senthilnathan: Is the validation done? We can close this and add docs for it if it's not already done /cc Arun Vijayraghavan

            keshav Keshav Murthy added a comment - Arunkumar Senthilnathan : Is the validation done? We can close this and add docs for it if it's not already done /cc Arun Vijayraghavan
            ingenthr Matt Ingenthron made changes -
            Sprint SDK48: …, [ 1307 ]
            jmorris Jeff Morris made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]

            People

              jmorris Jeff Morris
              keshav Keshav Murthy
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty