sunithabeeram opened a new pull request #3602: Cleanup: update code to return 
only physical columns for select * queries
URL: https://github.com/apache/incubator-pinot/pull/3602
 
 
   Addressing comments from https://github.com/apache/incubator-pinot/pull/3229 
as I am looking to fix https://github.com/apache/incubator-pinot/issues/3601.
   
   Changes in this PR include:
   - Adding isVirtualColumn to ColumnMetadata; this is not currently used, but 
included for completeness.
   - Updated code to use getPhysicalColumnNames instead of getColumnNames where 
appropriate.
   
   Testing done:
   - Added integration tests to cover the select * queries and assertions to 
ensure that virtual columns are not included.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to