[GitHub] [druid] gianm commented on pull request #11955: VirtualizedColumnSelectorFactory: Allow virtual columns to reference each other.

[GitBox <gi...@apache.org>]

gianm commented on pull request #11955:
URL: https://github.com/apache/druid/pull/11955#issuecomment-974565750


   @clintropolis with regard to whether virtual column implementations would handle this properly, I want to stress that virtual columns referencing other virtual columns is _already possible_ for cursors backed by regular segments. They do their own virtual column handling and it already works this way. The VirtualizedColumnSelectorFactory is updated in this patch to match that behavior.


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

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



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