deepthi912 opened a new pull request, #17882:
URL: https://github.com/apache/pinot/pull/17882

   - Added some more tests to make sure _previousKeyToRecordLocationMap is 
cleared after replacement and segment removal.
   - Changed the ERROR log to WARN when two consuming segments are present.
   - null check for previous record location makes sense only if the current 
segment for the key belongs to the consuming segment because the keys that 
point to current consuming segment's immutable segment will also have null 
previous record location once we replace those keys


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to