[GitHub] [pinot] jadami10 commented on pull request #10584: Enable index config by default.

["jadami10 (via GitHub)" <gi...@apache.org>]

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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org