richardstartin commented on PR #8712: URL: https://github.com/apache/pinot/pull/8712#issuecomment-1128047498
An upper bound is defined to be the smallest element greater than or equal to every member of a set, which is exactly what `getUpperBound` returns, similarly for `getLowerBound` so the names feel appropriate to me. There are much bigger, fundamental, challenges to adding range index support for `BigDecimal` than any confusion created by the method names. -- 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
