drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [drill] laurentgo opened a new pull request #2247: DRILL-7945: Stop patching newly added Guava methods
Date Fri, 04 Jun 2021 08:31:05 GMT

laurentgo opened a new pull request #2247:
URL: https://github.com/apache/drill/pull/2247


   # [DRILL-7945](https://issues.apache.org/jira/browse/DRILL-7945): Stop patching newly added
Guava methods
   
   ## Description
   
   In order to support Apache Iceberg, GuavaPatcher adds several methods to
   com.google.common.base.Preconditions required by Apache Iceberg runtime
   but not available in Guava 18.
   
   When Guava version was updated to 30.1.1, Guava Patcher tries to update
   existing methods with some code referencing package protected methods
   which do no exist anymore. Patching fails but as the methods do now
   exists, there's no compatibility issue. That said, GuavaPatcher emits
   some logging which might confuse end user.
   
   Remove the unnecessary patching of the Preconditions class and also add
   some unit test designed to fail if patching fails for any reason.
   
   
   ## Documentation
   N/A
   
   ## Testing
   Unit tests
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message