opensearch-project / search-processor

Search Request Processor: pipeline for transformation of queries and results inline with a search request.
Apache License 2.0
22 stars 24 forks source link

Personalized intelligent ranking for open search requests #138

Closed kulket closed 1 year ago

kulket commented 1 year ago

Description:

This change uses search pipeline feature of open search to allow customers to create a response processor. Response processor will rerank search results obtained from open search using Personalization solution before returning results to the user.

Check List:

Signed-off-by: Ketan Kulkarni kektnr@amazon.com

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

msfroh commented 1 year ago

@kulket -- Can you please rebase from main and force-push to your fork?

If you check the list of commits associated with this PR, it includes the Jackson-core update, which isn't actually part of this PR.

codecov[bot] commented 1 year ago

Codecov Report

Merging #138 (e695728) into main (2e78139) will increase coverage by 0.99%. The diff coverage is 84.27%.

@@             Coverage Diff              @@
##               main     #138      +/-   ##
============================================
+ Coverage     81.05%   82.05%   +0.99%     
- Complexity      218      293      +75     
============================================
  Files            29       41      +12     
  Lines           887     1131     +244     
  Branches        120      139      +19     
============================================
+ Hits            719      928     +209     
- Misses          106      127      +21     
- Partials         62       76      +14     
Impacted Files Coverage Δ
...search/search/relevance/SearchRelevancePlugin.java 0.00% <0.00%> (ø)
...izeintelligentranking/configuration/Constants.java 0.00% <0.00%> (ø)
...intelligentranking/requestparameter/Constants.java 0.00% <0.00%> (ø)
...zeintelligentranking/client/PersonalizeClient.java 58.82% <58.82%> (ø)
...uestparameter/PersonalizeRequestParameterUtil.java 75.00% <75.00%> (ø)
...ameter/PersonalizeRequestParametersExtBuilder.java 78.26% <78.26%> (ø)
...ng/reranker/impl/AmazonPersonalizedRankerImpl.java 82.05% <82.05%> (ø)
...ion/PersonalizeIntelligentRankerConfiguration.java 85.71% <85.71%> (ø)
...requestparameter/PersonalizeRequestParameters.java 88.00% <88.00%> (ø)
.../client/PersonalizeCredentialsProviderFactory.java 95.83% <95.83%> (ø)
... and 3 more
kulket commented 1 year ago

@kulket -- Can you please rebase from main and force-push to your fork?

If you check the list of commits associated with this PR, it includes the Jackson-core update, which isn't actually part of this PR.

Performed rebased as well as force push. I am seeing 3 commits now.