Issue33 #863
Closed
Issue33 #863
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…situations (opensearch-project#419) (opensearch-project#421) Signed-off-by: Petar Dzepina <[email protected]> (cherry picked from commit 76de976) Co-authored-by: Petar Dzepina <[email protected]>
ronnaksaxena
requested review from
bowenlan-amzn,
getsaurabh02,
lezzago,
praveensameneni,
xluo-aws,
gaobinlong,
Hailong-am,
amsiglan,
sbcd90,
eirsep and
AWSHurneyt
as code owners
…search-project#418) (opensearch-project#427) Signed-off-by: Mike Lee <[email protected]> (cherry picked from commit 8b476ca) Signed-off-by: Clay Downs <[email protected]> Co-authored-by: linuxboyng <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ensearch-project#439) (opensearch-project#442) Signed-off-by: Petar Dzepina <[email protected]> (cherry picked from commit 1f581dd) Co-authored-by: Petar Dzepina <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ensearch-project#435) (opensearch-project#444) * added support for mustache scripting of rollup.target_index field Signed-off-by: Petar Dzepina <[email protected]> * defekt fixes Signed-off-by: Petar Dzepina <[email protected]> * tests Signed-off-by: Petar Dzepina <[email protected]> * small refactor/improvements Signed-off-by: Petar Dzepina <[email protected]> * added wildcard check when creating rollup job; removed resolving targetIndex on Rollup init; added test for wildcards Signed-off-by: Petar Dzepina <[email protected]> * lint fixes Signed-off-by: Petar Dzepina <[email protected]> * moved target_index validation in getRollup resp handler Signed-off-by: Petar Dzepina <[email protected]> * added using toMap() Signed-off-by: Petar Dzepina <[email protected]> * removed catch block Signed-off-by: Petar Dzepina <[email protected]> * exception fix Signed-off-by: Petar Dzepina <[email protected]> * linter fix Signed-off-by: Petar Dzepina <[email protected]> * fixed IT fail Signed-off-by: Petar Dzepina <[email protected]> * added Exception catch block Signed-off-by: Petar Dzepina <[email protected]> (cherry picked from commit e858ab2) Co-authored-by: Petar Dzepina <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…opensearch-project#440) (opensearch-project#447) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <[email protected]> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <[email protected]> * reverted old error messages Signed-off-by: Petar Dzepina <[email protected]> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <[email protected]> * fixed sorting Signed-off-by: petar.dzepina <[email protected]> Co-authored-by: petar.dzepina <[email protected]> (cherry picked from commit b8a77d4) Co-authored-by: Petar Dzepina <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Add 2.2 release note Signed-off-by: Angie Zhang <[email protected]> * Add 2.2 release note Signed-off-by: Angie Zhang <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <[email protected]> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <[email protected]> * reverted old error messages Signed-off-by: Petar Dzepina <[email protected]> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <[email protected]> * fixed sorting Signed-off-by: petar.dzepina <[email protected]> * added ITs for multi rollup index search Signed-off-by: petardz <[email protected]> * added ITs for multi rollup index search#2 Signed-off-by: petardz <[email protected]> * detekt fixes Signed-off-by: petardz <[email protected]> * changed index names and rollup job Signed-off-by: petardz <[email protected]> * detekt fix Signed-off-by: petardz <[email protected]> * empty commit to trigger test pipeline Signed-off-by: petardz <[email protected]> Co-authored-by: petar.dzepina <[email protected]> (cherry picked from commit af5b366) Co-authored-by: Petar Dzepina <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ensearch-project#468) * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]> (cherry picked from commit 0f0ba16) Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: pgodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…pensearch-project#429) (opensearch-project#463) * bump version to 2.2 (opensearch-project#446) Signed-off-by: Ashish Agrawal <[email protected]> * Replica Count validation when awareness replica balance is enabled Signed-off-by: Gaurav Bafna <[email protected]> * Addressing PR comments Signed-off-by: Gaurav Bafna <[email protected]> Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: Gaurav Bafna <[email protected]> Co-authored-by: Ashish Agrawal <[email protected]> (cherry picked from commit f64c0c7) Co-authored-by: Gaurav Bafna <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
opensearch-project#485) (opensearch-project#486) * 483: Updated detekt plugin and snakeyaml dependency. Updated a code to reduce the number of issues after static analysis Signed-off-by: Stevan Buzejic <[email protected]> * 483: Updated snakeyaml version to use the latest Signed-off-by: Stevan Buzejic <[email protected]> Signed-off-by: Stevan Buzejic <[email protected]> (cherry picked from commit ed6ed10) Co-authored-by: Stevan Buzejic <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
opensearch-project#461) (opensearch-project#464) Signed-off-by: Khushboo Rajput <[email protected]> Signed-off-by: Khushboo Rajput <[email protected]> (cherry picked from commit 2f7d009) Co-authored-by: Khushboo Rajput <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…nsearch-project#487) * Transform maxclauses fix (opensearch-project#477) * transform maxClauses fix Signed-off-by: Petar Dzepina <[email protected]> * added bucket log to track processed buckets Signed-off-by: Petar Dzepina <[email protected]> * various renames/changes Signed-off-by: Petar Dzepina <[email protected]> * fixed detekt issues Signed-off-by: Petar Dzepina <[email protected]> * added comments to test Signed-off-by: Petar Dzepina <[email protected]> * removed debug logging Signed-off-by: Petar Dzepina <[email protected]> * empty commit to trigger checks Signed-off-by: Petar Dzepina <[email protected]> * reduced pageSize to 1 in few ITs to avoid flaky tests; fixed bug where pagesProcessed was calculated incorrectly Signed-off-by: Petar Dzepina <[email protected]> * reverted pagesProcessed change; fixed few ITs Signed-off-by: Petar Dzepina <[email protected]> Signed-off-by: Petar Dzepina <[email protected]> (cherry picked from commit 7475cfd) * Update TransformRunnerIT.kt Co-authored-by: Petar Dzepina <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…earch-project#471) (opensearch-project#488) (opensearch-project#107) Signed-off-by: Xuesong Luo <[email protected]> Signed-off-by: Xuesong Luo <[email protected]> (cherry picked from commit 39be4e3) Co-authored-by: xluo-aws <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…h-project#500) Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: bowenlan-amzn <[email protected]> (cherry picked from commit 50ac1e9) Co-authored-by: bowenlan-amzn <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…roject#502) Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: bowenlan-amzn <[email protected]> (cherry picked from commit a9d04ec) Co-authored-by: bowenlan-amzn <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Increment version to 2.3.0-SNAPSHOT Signed-off-by: opensearch-ci-bot <[email protected]> * bump mockito-core to 4.7.0, httpcore to 4.4.15 Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: bowenlan-amzn <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: bowenlan-amzn <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Update 2.3 release note Signed-off-by: Angie Zhang <[email protected]> * Update 2.3 release note Signed-off-by: Angie Zhang <[email protected]> * Update 2.3 release note Signed-off-by: Angie Zhang <[email protected]> * Update 2.3 release note Signed-off-by: Angie Zhang <[email protected]> * Update 2.3 release note Signed-off-by: Angie Zhang <[email protected]> Signed-off-by: Angie Zhang <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ect#528) (opensearch-project#529) * Revert Disable detekt: 50ac1e9 Signed-off-by: Siddhant Deshmukh <[email protected]> * Remove force choosing snakeyml 1.31 Signed-off-by: Siddhant Deshmukh <[email protected]> * Force snakeyaml 1.32 Signed-off-by: Siddhant Deshmukh <[email protected]> * Empty commit Signed-off-by: Siddhant Deshmukh <[email protected]> Signed-off-by: Siddhant Deshmukh <[email protected]> (cherry picked from commit 943c32b) Co-authored-by: Siddhant Deshmukh <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…rch-project#785) * recursion removal from AddPolicy Action Signed-off-by: Petar Dzepina <[email protected]> * added test Signed-off-by: Petar Dzepina <[email protected]> * debug logging Signed-off-by: Petar Dzepina <[email protected]> * removed single node testcase Signed-off-by: Petar Dzepina <[email protected]> * added security test; fixed index permission check Signed-off-by: Petar Dzepina <[email protected]> * test fix Signed-off-by: Petar Dzepina <[email protected]> * addressing comments Signed-off-by: Petar Dzepina <[email protected]> * test cleanup Signed-off-by: Petar Dzepina <[email protected]> * reverted security inject changes Signed-off-by: Petar Dzepina <[email protected]> * fixed weak password error when creating test user Signed-off-by: Petar Dzepina <[email protected]> * test tweak Signed-off-by: Petar Dzepina <[email protected]> --------- Signed-off-by: Petar Dzepina <[email protected]> (cherry picked from commit 45b1476) Signed-off-by: Ronnak Saxena <[email protected]>
opensearch-project#791) * CRUD api for long running operation notification config (opensearch-project#722) * notification crud api Signed-off-by: zhichao-aws <[email protected]> * changes based on comments; add tests Signed-off-by: zhichao-aws <[email protected]> * changes for comment Signed-off-by: zhichao-aws <[email protected]> * change behavior for get action index not exists, fix test cases Signed-off-by: zhichao-aws <[email protected]> * fix weak password for changes in security plugin Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: zhichao-aws <[email protected]> (cherry picked from commit 96f7380) * fix test code apache http package version Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: zhichao-aws <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ct#793) * Feature/index operation notification (opensearch-project#712) * Feature/index operation notification (opensearch-project#707) * task completion listener for long running task Signed-off-by: Hailong Cui <[email protected]> * task completion listener for long running task Signed-off-by: Hailong Cui <[email protected]> * remove unused import Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> * index action, no validation&test Signed-off-by: zhichao-aws <[email protected]> get action. no validation&test Signed-off-by: zhichao-aws <[email protected]> index and get action (draft) changes for rebase main Signed-off-by: zhichao-aws <[email protected]> index and get, no validation&test Signed-off-by: zhichao-aws <[email protected]> * refactor on package name Signed-off-by: Hailong Cui <[email protected]> * +delete, change index Signed-off-by: zhichao-aws <[email protected]> * Implement notification with ActionFilter Signed-off-by: Hailong Cui <[email protected]> * Query notification channel Signed-off-by: Hailong Cui <[email protected]> * Notification message update Signed-off-by: Hailong Cui <[email protected]> * new version of crud Signed-off-by: zhichao-aws <[email protected]> * validate task id and action name; unitfy taskId Signed-off-by: zhichao-aws <[email protected]> * Add unit test case Signed-off-by: Hailong Cui <[email protected]> * refactor on response parser Signed-off-by: Hailong Cui <[email protected]> * fix unit test case Signed-off-by: Hailong Cui <[email protected]> * modify code structure, change ID to Id Signed-off-by: zhichao-aws <[email protected]> * add some test, delete message template Signed-off-by: zhichao-aws <[email protected]> * Add integration test Signed-off-by: Hailong Cui <[email protected]> * Add integration test for action listener Signed-off-by: Hailong Cui <[email protected]> * remove refresh policy in request Signed-off-by: zhichao-aws <[email protected]> * add some tests, fix actionfilter refresh parameter Signed-off-by: zhichao-aws <[email protected]> * Remove duplicate errors for reindex Signed-off-by: Hailong Cui <[email protected]> * add UT and IT Signed-off-by: zhichao-aws <[email protected]> * fix some code style Signed-off-by: zhichao-aws <[email protected]> * enable com.sun.net.httpserver.* for test Signed-off-by: Hailong Cui <[email protected]> * change index name, fix bug Signed-off-by: zhichao-aws <[email protected]> * modifications after meeting Signed-off-by: zhichao-aws <[email protected]> * don't continue wait for shard to be started for async mode Signed-off-by: Hailong Cui <[email protected]> * handle IndexNotFoundException for sending notification Signed-off-by: Hailong Cui <[email protected]> * add security test case Signed-off-by: zhichao-aws <[email protected]> * add security test case Signed-off-by: zhichao-aws <[email protected]> * refactor on file names Signed-off-by: Hailong Cui <[email protected]> * add success/failure condition Signed-off-by: zhichao-aws <[email protected]> * filter channels by conditions Signed-off-by: Hailong Cui <[email protected]> * Remove unused field enable for index .opensearch-admin-panel Signed-off-by: Hailong Cui <[email protected]> * add more integration testcase Signed-off-by: Hailong Cui <[email protected]> * change admin panel to control center Signed-off-by: zhichao-aws <[email protected]> * Update mock server address to localhost for docker env Signed-off-by: Hailong Cui <[email protected]> * remove unused field in response Signed-off-by: zhichao-aws <[email protected]> * update integ test Signed-off-by: Hailong Cui <[email protected]> * remove default for all Signed-off-by: zhichao-aws <[email protected]> * Remove default for all Signed-off-by: Hailong Cui <[email protected]> * remove filter by backend roles Signed-off-by: zhichao-aws <[email protected]> * fix build Signed-off-by: zhichao-aws <[email protected]> * optimize constructor code Signed-off-by: zhichao-aws <[email protected]> * add request doc id validation Signed-off-by: zhichao-aws <[email protected]> * merge get and search Signed-off-by: zhichao-aws <[email protected]> * remove get before delete Signed-off-by: zhichao-aws <[email protected]> * add node id validation Signed-off-by: zhichao-aws <[email protected]> * auto create doc for update Signed-off-by: zhichao-aws <[email protected]> * fix test case Signed-off-by: zhichao-aws <[email protected]> * delete restUpdateAction, fix test Signed-off-by: zhichao-aws <[email protected]> * add dry run option Signed-off-by: zhichao-aws <[email protected]> * fix bug, add test case Signed-off-by: zhichao-aws <[email protected]> * add xcontent tests Signed-off-by: zhichao-aws <[email protected]> * notification crud api Signed-off-by: zhichao-aws <[email protected]> * changes based on comments; add tests Signed-off-by: zhichao-aws <[email protected]> * changes for comment Signed-off-by: zhichao-aws <[email protected]> * change behavior for get action index not exists, fix test cases Signed-off-by: zhichao-aws <[email protected]> * fix merge Signed-off-by: zhichao-aws <[email protected]> * delay 5s for sending runtime notification Signed-off-by: Hailong Cui <[email protected]> * update log message Signed-off-by: Hailong Cui <[email protected]> * delay 5s for runtime notification policy Signed-off-by: Hailong Cui <[email protected]> * Update notification message Signed-off-by: Hailong Cui <[email protected]> * filter duplicate channel Signed-off-by: Hailong Cui <[email protected]> * exception handling Signed-off-by: Hailong Cui <[email protected]> * Add cluster name into title Signed-off-by: Hailong Cui <[email protected]> * fix UT for failure messagee Signed-off-by: Hailong Cui <[email protected]> * include cluster name into notification messag Signed-off-by: Hailong Cui <[email protected]> * fix UT failures Signed-off-by: Hailong Cui <[email protected]> * fix detekt issue Signed-off-by: Hailong Cui <[email protected]> * fix NPE Signed-off-by: Hailong Cui <[email protected]> * force merge message Signed-off-by: Hailong Cui <[email protected]> * remove duplicate error of reindex Signed-off-by: Hailong Cui <[email protected]> * notification message rewording Signed-off-by: Hailong Cui <[email protected]> * notification message wording change and unit test case Signed-off-by: Hailong Cui <[email protected]> * update block write wording Signed-off-by: Hailong Cui <[email protected]> * Add more unit test case Signed-off-by: Hailong Cui <[email protected]> * Add more logs Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: zhichao-aws <[email protected]> (cherry picked from commit 89bd6de) * Fix backport failure Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Co-authored-by: Hailong Cui <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Add 2.8 release notes Signed-off-by: Xuesong Luo <[email protected]> (cherry picked from commit d921f0b) Co-authored-by: Xuesong Luo <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
opensearch-project#797) * Use original PR numbers in 2.8 release notes. Signed-off-by: Xuesong Luo <[email protected]> --------- Signed-off-by: Xuesong Luo <[email protected]> (cherry picked from commit 104c4b8) Co-authored-by: Xuesong Luo <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Support for system index interface Signed-off-by: bowenlan-amzn <[email protected]> * Handle rejected execution exception as retryable in retry block Signed-off-by: bowenlan-amzn <[email protected]> * Fix tests Signed-off-by: bowenlan-amzn <[email protected]> * Fix tests Signed-off-by: bowenlan-amzn <[email protected]> --------- Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
opensearch-project#803) * 202: Added format property when specifying the date histogram Signed-off-by: Stevan Buzejic <[email protected]> * 202: Added component responsible for building the target index mapping once the transform is being triggered. Signed-off-by: Stevan Buzejic <[email protected]> * 202: date_histogram considered in the case of the creating the target index for the date fields when transform is executed Signed-off-by: Stevan Buzejic <[email protected]> * 202: Enabled target index date field mappings if those fields are used in aggregations or as a term aggregation for defining the buckets Signed-off-by: Stevan Buzejic <[email protected]> * Updated code according to comments. Added targetIndexMapping when transform preview action is triggered Signed-off-by: Stevan Buzejic <[email protected]> * Updated schema versions Signed-off-by: Stevan Buzejic <[email protected]> * Addressed the comments Signed-off-by: Stevan Buzejic <[email protected]> * Refactored transform tests related with aggregation based on a date field. Updated transform preview action to consider target index mapping when using a date field. Kept formatting of the date field in target index. Signed-off-by: Stevan Buzejic <[email protected]> * detekt fix Signed-off-by: Stevan Buzejic <[email protected]> * Added zone in IT Signed-off-by: Stevan Buzejic <[email protected]> * Added function for creating target index mapping that considers transform mapping json. Target date field mappings are generated after transform validation when running transform. Removed target index date field values formatting. emoved default format for date_histogram because of the rollup. Updated schema version in test. Signed-off-by: Stevan Buzejic <[email protected]> --------- Signed-off-by: Stevan Buzejic <[email protected]> (cherry picked from commit 42833b1) Co-authored-by: Stevan Buzejic <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…pensearch-project#811) * Added myself to the repo. * Added myself to the repo. * Added maintainers to the repo. * Update MAINTAINERS.md * Update MAINTAINERS.md --------- Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: bowenlan-amzn <[email protected]> Co-authored-by: bowenlan-amzn <[email protected]> Co-authored-by: Sayali Gaikawad <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
* Metadata members from ImmutableOpenMap to j.u.Map Signed-off-by: Hailong Cui <[email protected]> * add field remoteStoreIndexShallowCopy in SM TestUtils (opensearch-project#825) Signed-off-by: zhichao-aws <[email protected]> * fix NPE for transform aggregations (opensearch-project#830) Signed-off-by: Hailong Cui <[email protected]> * Fix detekt Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: zhichao-aws <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…project#831) (opensearch-project#842) * fix LRON unwrap exception and some IT Signed-off-by: zhichao-aws <[email protected]> * remove useless log Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: Hailong Cui <[email protected]> (cherry picked from commit f07fa2b) Co-authored-by: zhichao-aws <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…arch-project#843) Signed-off-by: Hailong Cui <[email protected]> (cherry picked from commit 2a9bcd6) Co-authored-by: Hailong Cui <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…arch-project#847) (opensearch-project#848) Signed-off-by: bowenlan-amzn <[email protected]> (cherry picked from commit 77e8fd0) Co-authored-by: bowenlan-amzn <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
…ch-project#786) (opensearch-project#845) (cherry picked from commit 30e16d9) Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
…ot sure if it seeing all indices Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
Signed-off-by: Ronnak Saxena <[email protected]>
|
you can raise PR to 2.x or cherry-pick this commit this commit to main branch |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue 33
Description of changes:
•Abstracted logic to handle nonidempotent failures that could cause inapropriate error log "Previous action was not able to update IndexMetaData."
•Detects transient failure for rollover step with testing
CheckList:
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.