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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
