jadami10 commented on PR #10584: URL: https://github.com/apache/pinot/pull/10584#issuecomment-1501903726
Thank you for the awesome test coverage. I'll let others review as well. In case there's some concern with always enabling an index unless the config is exactly `"enabled": false`, I definitely think the way you did it here is the least surprising for users. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
