You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2019/11/05 02:33:51 UTC
[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4774: Refactor
virtualColumn to extraColumn
Jackie-Jiang commented on issue #4774: Refactor virtualColumn to extraColumn
URL: https://github.com/apache/incubator-pinot/pull/4774#issuecomment-549636862
The current definition of virtual column is the column backed by the VirtualColumnProvider class. VirtualColumnProvider can store physical resources/access remote resources if necessary. If this is not changed, I don't think we need to rename the virtual column to extra column. IMO, extra column is not as intuitive as virtual column.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org