[
https://issues.apache.org/jira/browse/DRILL-6633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16592630#comment-16592630
]
ASF GitHub Bot commented on DRILL-6633:
---------------------------------------
vrozov commented on issue #1397: DRILL-6633: Replace usage of Guava classes by
JDK ones
URL: https://github.com/apache/drill/pull/1397#issuecomment-415977809
@arina-ielchiieva / @vvysotskyi I don't see a benefit of blindly replacing
Guava classes/methods with JDK ones that provide equivalent or similar
functionality. It should be sufficient to replace deprecated and unstable
classes/methods. So, even though an enormous amount of work was done, I don't
see how Drill and drill dev community benefit from that work. I already
mentioned that in my review comments and also on the mailing list. IMO, it will
be more beneficial for Drill customers to resolve DRILL-6422 and #1264.
@arina-ielchiieva:
- I don't share your concern with PR being open for a month. In Apache PR
may be open for much longer and there are multiple examples of PRs being open
for 2, 3 and more month in Drill and other Apache projects.
- I agree that it is not only possible but it is much better not to fix all
problems in one PR. In some cases, instead of a follow up JIRA that may never
be fixed, I'd prefer to see a pre-cursor JIRA and PR and this PR is an example
where I'd like to see several Drill classes being removed or refactored first
(due to a larger benefit that I see for removing those classes compared to
removing Guava classes).
----------------------------------------------------------------
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]
> Replace usage of Guava classes by JDK ones
> ------------------------------------------
>
> Key: DRILL-6633
> URL: https://issues.apache.org/jira/browse/DRILL-6633
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.14.0
> Reporter: Volodymyr Vysotskyi
> Assignee: Volodymyr Vysotskyi
> Priority: Major
> Fix For: 1.15.0
>
>
> Currently, Drill uses classes from Guava which can be replaced after moving
> to JDK 8.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
