[GitHub] [pinot] Jackie-Jiang commented on pull request #7912: Revert "replication factor to be obtained from tenant configuration in case of dimension table"

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

Jackie-Jiang commented on pull request #7912:
URL: https://github.com/apache/pinot/pull/7912#issuecomment-995401340


   The problem is from the meaning of the storage quota value, which counts storage for all replicas. E.g. Storage quota of 30G for a table with 3 replicas allow 10G segments per replica.
   For a dimension table, the change is actually trying to apply the same logic, where storage for all replicas are counted. Without the change, storage quota for dimension table with different replicas has different per replica limit, but replication does not really apply to dimension table.
   
   I feel a clean solution would be introducing the per replica storage quota. Add @jackjlli here who contributed the storage quota feature. 


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