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

fix: refresh the rollup index as part of the rollup indexer actions #86992

Commits on May 20, 2022

  1. fix: move refresh just before the match_all query

    Using the command line provided in elastic#81983 I could not reproduce the issue.
    Anyway, changing the timeout from 5 seconds to just 1 second I could
    reproduce the issue. At the end, keeping a timeout of 1 second I could fix
    the issue moving the refresh operation just before running the match_all
    query. My understanding is that the timeout is used to give the rollup job
    enough time to complete the rollup operation. Anyway, other than that, after
    the rollup job has written data into the rollup index we need to refresh it
    in order for the subsequent search operation to hit the expected rolledup
    document.
    salvatore-campagna committed May 20, 2022
    Configuration menu
    Copy the full SHA
    7968c70 View commit details
    Browse the repository at this point in the history

Commits on May 31, 2022

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

Commits on Jun 1, 2022

  1. Revert "fix: move refresh just before the match_all query"

    Instead of refreshing from the yaml test we refresh once the
    indexing operation is complete in the callback.
    
    This reverts commit 7968c70.
    salvatore-campagna committed Jun 1, 2022
    Configuration menu
    Copy the full SHA
    d3e9dd1 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    6385514 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2022

  1. Configuration menu
    Copy the full SHA
    f48593c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ea442d5 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    2688bb1 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    8e11d8c View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2022

  1. Configuration menu
    Copy the full SHA
    a46af31 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f9bbc68 View commit details
    Browse the repository at this point in the history
  3. fix: just mock the search action call

    The client is used to call the refresh action too. As a result, using
    'any()' as the action would match both the search and refresh actions
    while we just need to handle the search action.
    salvatore-campagna committed Jun 9, 2022
    Configuration menu
    Copy the full SHA
    691c54a View commit details
    Browse the repository at this point in the history

Commits on Jul 5, 2022

  1. Configuration menu
    Copy the full SHA
    3bd3a94 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c1f70f3 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    4e5fc42 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    16bd5b0 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    24ee30a View commit details
    Browse the repository at this point in the history
  6. fix: move listener callback into the refresh listener

    We also need to mock the refresh response for the client
    to actually propagate the result of the refresh call.
    salvatore-campagna committed Jul 5, 2022
    Configuration menu
    Copy the full SHA
    ceb5c6b View commit details
    Browse the repository at this point in the history