vrajat opened a new pull request, #16102: URL: https://github.com/apache/pinot/pull/16102
#15773 added support for logical tables in controller query API. However the support broke queries that referenced hybrid tables. This PR fixes bug by simplifying the loop. Previously there were two sets of two loops - one to find tables and another to find logical tables. Two loops made it hard to find out if tables were found or not. In the fix, only a single loop on table names is used to check for both physical and logical tables. Another issue was that there were no tests to check changes in this part of the code. A couple of tests have been added to test controller query API. Closes #16032 -- 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
