richardstartin opened a new pull request, #8709: URL: https://github.com/apache/pinot/pull/8709
Pinot has two `Pair` types: one in `common` and one in `pinot-spi`. * These types are inconsistent with primitive pair types in `Pairs` - they have different method names to access the values * They are constructed inconsistently throughout the codebase, leading to many IDE warnings when type parameters are omitted. It's not possible to construct an apache commons `Pair` without the types being inferred, so this simple change will improve future code quality as new uses of `Pair` are added. -- 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
