-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Compare strings with equals() instead of == #16415
Conversation
Signed-off-by: Dmitry Kryukov <[email protected]>
Signed-off-by: Dmitry Kryukov <[email protected]>
❌ Gradle check result for d138929: 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? |
server/src/main/java/org/opensearch/search/suggest/phrase/NoisyChannelSpellChecker.java
Outdated
Show resolved
Hide resolved
test/framework/src/main/java/org/opensearch/index/shard/IndexShardTestCase.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Dmitry Kryukov <[email protected]>
❌ Gradle check result for 28de794: 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? |
DCO is failing because the email extension in your "Author" information doesn't match the email extension in your signoff. Can you correct that please? |
Will continue here #16461 |
Previous PR #15369
(rebase caused mess again)