Akanksha-kedia commented on PR #18963: URL: https://github.com/apache/pinot/pull/18963#issuecomment-5087033081
Hi @Jackie-Jiang — yes! Tests for `FSTIndexHandler` are being added in the companion PR #18960. This PR (#18963) focuses specifically on `IFSTIndexHandler`. Both PRs together close the coverage gap for the FST handler family. CI is passing and this has been approved by @gortiz — would appreciate your review/merge when you get a chance! -- 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]
