You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/06/19 09:40:51 UTC
[GitHub] [drill] pjfanning opened a new pull request, #2577: Upgrade hadoop 2 due to CVE
pjfanning opened a new pull request, #2577:
URL: https://github.com/apache/drill/pull/2577
# [DRILL-XXXX](https://issues.apache.org/jira/browse/DRILL-XXXX): PR Title
(Please replace `PR Title` with actual PR Title)
## Description
(Please describe the change. If more than one ticket is fixed, include a reference to those tickets.)
## Documentation
(Please describe user-visible changes similar to what should appear in the Drill documentation.)
## Testing
(Please describe how this PR has been tested.)
--
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: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] pjfanning commented on pull request #2577: DRILL-8251: Upgrade hadoop 2 due to CVE
Posted by GitBox <gi...@apache.org>.
pjfanning commented on PR #2577:
URL: https://github.com/apache/drill/pull/2577#issuecomment-1160127175
@cgivre thanks - marked as ready to review
--
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: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] jnturton commented on pull request #2577: DRILL-8251: Upgrade hadoop 2 due to CVE
Posted by GitBox <gi...@apache.org>.
jnturton commented on PR #2577:
URL: https://github.com/apache/drill/pull/2577#issuecomment-1159757273
> LGTM +1
>
> @jnturton Is there anything special we need to do for this before we merge?
When we update our Hadoop 3 libs we must also update the winutils binaries we ship for Windows. But in the case of Hadoop 2 we do not support Windows, and I can't imagine anyone out there who wants that, so LGTM.
--
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: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] cgivre commented on pull request #2577: DRILL-8251: Upgrade hadoop 2 due to CVE
Posted by GitBox <gi...@apache.org>.
cgivre commented on PR #2577:
URL: https://github.com/apache/drill/pull/2577#issuecomment-1159889030
@pjfanning It looks like you have two approvals. If you want to mark this as ready for review, we can merge. Thanks for this!
--
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: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] cgivre merged pull request #2577: DRILL-8251: Upgrade Hadoop 2 Due to CVE
Posted by GitBox <gi...@apache.org>.
cgivre merged PR #2577:
URL: https://github.com/apache/drill/pull/2577
--
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: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org