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 2021/06/18 21:18:58 UTC

[GitHub] [incubator-pinot] mayankshriv commented on pull request #7071: Clear legacy configs when converting to new TableConfig.

mayankshriv commented on pull request #7071:
URL: https://github.com/apache/incubator-pinot/pull/7071#issuecomment-863612278


   > We should convert the config based on the table type. For OFFLINE table, convert the batch config; for REALTIME table, convert the stream config.
   > Or only convert the field if the deprecated field is set
   
   Updated with the latter suggestion to convert only if deprecated field is set.


-- 
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



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