gf2121 commented on code in PR #12699: URL: https://github.com/apache/lucene/pull/12699#discussion_r1391060906
########## lucene/core/src/java/org/apache/lucene/codecs/lucene90/blocktree/IntersectTermsEnumFrame.java: ########## @@ -142,12 +138,20 @@ public void setState(int state) { } void load(BytesRef frameIndexData) throws IOException { - if (frameIndexData != null) { Review Comment: Yes. We checked null before for [line 119](https://github.com/apache/lucene/pull/12699/files/71ef88acf5bc3b2c47086b55d63d64720b4232b7#diff-77d427360dcf008af183201e0e748d77ccadae1b2359fa96940c407b48b3f85bR119). It is cast to long now and we do the null check in [line 153](https://github.com/apache/lucene/pull/12699/files/71ef88acf5bc3b2c47086b55d63d64720b4232b7#diff-77d427360dcf008af183201e0e748d77ccadae1b2359fa96940c407b48b3f85bR153) instead. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org