NIFI-13186 corrected required runtime dep lucene-backward-codecs and …#8782
NIFI-13186 corrected required runtime dep lucene-backward-codecs and …#8782joewitt wants to merge 1 commit intoapache:mainfrom
Conversation
|
Validating against m2 built index (lucene 8.11.3) vs this PR. Will share more details. |
…updated tests Updated references to lucene-9 indexes and removed legacy old style indexes.
|
Additional code changes applied now. Tested with a previous build generating Lucene-8 indexes with a range of content. Verified then again post this patch and it picks up where it left off, is still queryable, and continues generating new indexes with the lucene-9 naming. Was able to query both old and new data in the system. Also verifed query syntax still works despite not including the queryparsers module at this point. |
exceptionfactory
left a comment
There was a problem hiding this comment.
Thanks for addressing Lucene compatibility with this dependency and code changes @joewitt. I noted two questions regarding particular library removals and one method removal, but otherwise the changes seem straightforward.
...nance-repository/src/main/java/org/apache/nifi/provenance/index/lucene/LuceneEventIndex.java
Show resolved
Hide resolved
- Updated references to lucene-9 indexes and removed legacy old style indexes This closes apache#8782 Signed-off-by: David Handermann <exceptionfactory@apache.org>i
- Updated references to lucene-9 indexes and removed legacy old style indexes This closes apache#8782 Signed-off-by: David Handermann <exceptionfactory@apache.org>i
…updated tests
Summary
NIFI-13186
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking
Pull Request Tracking
NIFI-00000NIFI-00000Pull Request Formatting
mainbranchVerification
Please indicate the verification steps performed prior to pull request creation.
Build
mvn clean install -P contrib-checkUI Contributions
Licensing
LICENSEandNOTICEfilesDocumentation