[GitHub] [pinot] KKcorps commented on pull request #9861: Allow uploading realtime segments via CLI

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

KKcorps commented on PR #9861:
URL: https://github.com/apache/pinot/pull/9861#issuecomment-1336335004

   Done. but the reason I didn't want to make it backward compatible is this TODO comment in the existing /v2/segments code
   ```json
           // TODO: remove this when we completely deprecate the table name from segment metadata
           rawTableName = segmentMetadata.getTableName();
           LOGGER.warn("Table name is not provided as request query parameter when uploading segment: {} for table: {}",
               segmentName, rawTableName);
    ```           


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

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