Skip to content

Conversation

@johnnyshields
Copy link

@johnnyshields johnnyshields commented Oct 24, 2025

  • This PR is based on 7.16.2 #27
  • I've renamed the plugin to analysis-openkoreantext (was elasticsearch-analysis-openkoreantext, which breaks convention with analysis-nori, analysis-kuromoji, etc.)
  • The Lucene test framework in ES 9.x enforces a strict naming convention for tests in the org.apache.lucene package. Test classes must start with Test rather than end with Test, hence the file renames in this PR.

  • 이 PR은 7.16.2 #27 을 베이스로 하여 구현되었습니다.
  • 플러그인 이름을 analysis-openkoreantext로 변경했습니다.
    기존 이름은 elasticsearch-analysis-openkoreantext였지만, 이는 analysis-nori, analysis-kuromoji 등과의 명명 규칙에 맞지 않았습니다.
  • ES 9.x의 Lucene 테스트 프레임워크에서는 org.apache.lucene 패키지 내 테스트에 대해 엄격한 명명 규칙이 적용됩니다. 테스트 클래스 이름은 Test로 끝나는 것이 아니라 Test로 시작해야 합니다. 이 PR에서 파일 이름을 변경한 이유가 바로 그것입니다.

@johnnyshields johnnyshields changed the title Upgrade Elasticsearch to 9.1.4 Upgrade Elasticsearch to 9.1.4 + fix plugin name Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants