Jackie-Jiang commented on issue #3725: Replace partition ranges with partitions URL: https://github.com/apache/incubator-pinot/pull/3725#issuecomment-455759857 @kishoreg Currently we are storing a list of ranges, which is quite redundant (and not supported when range covers more than 1 value). A list of integer partitions should be good enough, clear and simple. We should not have large number of partitions, so ranges do not buy us much. This change is mostly for simplicity and readability, which is not a must have, but a good to have IMO. With this, it's easier to add support for multi-partitions in the future.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
