Make multiple settings dynamic for Merge on Flush #17495
Conversation
kkewwei
requested review from
Bukhtawar,
CEHENKLE,
Rishikesh1159,
VachaShah,
anasalkouz,
andrross,
ashking94,
bugmakerrrrrr,
cwperks,
dblock,
dbwiddis,
gbbafna,
jainankitk,
jed326,
kotwanikunal,
linuxpi,
mch2,
msfroh,
nknize,
owaiskazi19,
reta,
sachinpkale,
saratvemulapalli,
shwetathareja and
sohami
as code owners
github-actions
bot
added
bug
kkewwei
force-pushed
the
dynamic_settings
branch
from
6adb78d to
bbb906f
Compare
|
❌ Gradle check result for bbb906f: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❕ Gradle check result for cf7e975: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
|
❌ Gradle check result for c77e86d: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
IndexActionIT.testAutoGenerateIdNoDuplicates #16576 |
kkewwei
force-pushed
the
dynamic_settings
branch
from
c77e86d to
cb4b0f1
Compare
|
❌ Gradle check result for cb4b0f1: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 11587de: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
@msfroh Could you please continue to review in your spare time? Thank you very much. |
server/src/main/java/org/opensearch/index/engine/InternalEngine.java
Outdated
Show resolved
Hide resolved
…lush_merge_wait_time`, `index.merge_on_flush.policy`, `index.check_pending_flush.enabled` dynamic Signed-off-by: kkewwei <[email protected]> Signed-off-by: kkewwei <[email protected]>
Signed-off-by: kkewwei <[email protected]>
|
❌ Gradle check result for ab353dc: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
This PR is stalled because it has been open for 30 days with no activity. |
3 participants
Description
index.merge_on_flush.enabledindex.merge_on_flush.max_full_flush_merge_wait_timeindex.merge_on_flush.policyindex.check_pending_flush.enabled.Related Issues
Resolves #10048
Check List
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.