Skip to content

Conversation

@l-trotta
Copy link
Contributor

As reported in elastic/elasticsearch-java#1082, mode in KuromojiAnalyzer is nullable, the server gets it from settings if not present -> server code

@github-actions
Copy link
Contributor

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
get 🟢 317/317 → 321/321 314/314 → 318/318
index 🟢 1440/1440 → 1445/1445 1442/1442 → 1447/1447
indices.create 🔴 1373/1397 → 1378/1402 1397/1397 → 1402/1402
indices.get_mapping 🔴 198/198 → 202/202 187/198 → 191/202
search 🔴 2584/2611 → 2585/2612 2611/2611 → 2612/2612

You can validate these APIs yourself by using the make validate target.

@l-trotta l-trotta merged commit 9b51ce2 into main Oct 20, 2025
16 checks passed
@l-trotta l-trotta deleted the kuromoji-fix branch October 20, 2025 09:57
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
l-trotta added a commit that referenced this pull request Oct 20, 2025
(cherry picked from commit 9b51ce2)

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Oct 20, 2025
(cherry picked from commit 9b51ce2)

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Oct 20, 2025
(cherry picked from commit 9b51ce2)

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Oct 20, 2025
(cherry picked from commit 9b51ce2)

Co-authored-by: Laura Trotta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants