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

Bug fixes and refactoring #38

Merged
merged 3 commits into from
Dec 19, 2022

Commits on Dec 17, 2022

  1. Bug fixes and refactoring

    1. Shallow clone SearchResponse in SearchActionFilter. The only
       properties that we're currently changing are SearchHit and
       timeTookMillis. This is also less brittle as we don't try to
       implement deserialization ourselves.
    2. KendraIntelligentRanker doesn't transform if the required client
       settings (endpoint + execution plan ID) are missing.
    3. A lot of refactoring to prepare for other transformers. Where we
       previously had logic of the form `if kendra_intelligent_ranking then
       do Kendra intelligent ranking`, now the APIs support arbitrary named
       transformers. This required updates to tests and configuration code.
    4. Added tests and fixes for SearchConfigurationExtBuilder.
    
    Signed-off-by: Michael Froh <froh@amazon.com>
    msfroh committed Dec 17, 2022
    Configuration menu
    Copy the full SHA
    ddb2bfd View commit details
    Browse the repository at this point in the history
  2. Address PR comments from myself

    Looking at the PR on GitHub, I spotted a few mistakes and possible
    improvements.
    
    Signed-off-by: Michael Froh <froh@amazon.com>
    msfroh committed Dec 17, 2022
    Configuration menu
    Copy the full SHA
    258cde4 View commit details
    Browse the repository at this point in the history
  3. Address comments by noCharger

    Signed-off-by: Michael Froh <froh@amazon.com>
    msfroh committed Dec 17, 2022
    Configuration menu
    Copy the full SHA
    698f275 View commit details
    Browse the repository at this point in the history