Skip to content

Commit

Permalink
Updated the refresh to be true as that is what it appears what it was…
Browse files Browse the repository at this point in the history
… changing into before
  • Loading branch information
FrankHassanabad committed Jul 19, 2021
1 parent 3becbed commit 99fabce
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,9 @@ export const setSignalsStatusRoute = (router: SecuritySolutionPluginRouter) => {
index: siemClient.getSignalsIndex(),
conflicts: conflicts ?? 'abort',
// https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-update-by-query.html#_refreshing_shards_2
// Note: We cannot use "refresh: wait_for" here.
// Note: Before we tried to use "refresh: wait_for" but I do not think that was available and instead it defaulted to "refresh: true"
// but the tests do not pass with "refresh: false". If t some point a "refresh: wait_for" is implemented, we should use that instead.
refresh: true,
body: {
script: {
source: `ctx._source.signal.status = '${status}'`,
Expand Down

0 comments on commit 99fabce

Please sign in to comment.