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

Remove InternalSearchResponse as well as most usage of SearchResponseSections #103583

Commits on Dec 19, 2023

  1. Remove InternalSearchResponse as well as most usage of SearchResponse…

    …Sections
    
    This indirection isn't really necessary now that its use in the high level rest client went away.
    -> flattened the `SearchResponse` class to make ref counting easier
    -> removed `InternalSearchResponse` whose functionality was completely redundant now
    -> removed serialization and most of the usage of `SearchResponseSections`, only keeping this for
    the few search execution spots that currently use it and where removing it won't get us anything and
    will be a huge changeset.
    original-brownbear committed Dec 19, 2023
    Configuration menu
    Copy the full SHA
    2ec8707 View commit details
    Browse the repository at this point in the history
  2. fix test

    original-brownbear committed Dec 19, 2023
    Configuration menu
    Copy the full SHA
    638e366 View commit details
    Browse the repository at this point in the history

Commits on Dec 20, 2023

  1. Configuration menu
    Copy the full SHA
    e628105 View commit details
    Browse the repository at this point in the history