abhishekbafna commented on PR #15720: URL: https://github.com/apache/pinot/pull/15720#issuecomment-2861751682
> Should `LogicalTableUtils` be renamed to `LogicalTableConfigUtils` ? > > Also is there a test to make sure query config is stored and can be retrieved ? Yes. PinotLogicalTableResourceTest#verifyLogicalTableExists compares the logical table config object with GET logical table API response. So it covers all the configuration for the logical table. -- 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
