You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by "pjfanning (via GitHub)" <gi...@apache.org> on 2023/06/24 10:01:34 UTC

[PR] [DRILL-8443] Upgrade netty due to CVE (drill)

pjfanning opened a new pull request, #2813:
URL: https://github.com/apache/drill/pull/2813

   ## Description
   
   https://github.com/apache/drill/security/dependabot/45
   
   ## Testing
   
   CI build


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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "vvysotskyi (via GitHub)" <gi...@apache.org>.
vvysotskyi commented on PR #2813:
URL: https://github.com/apache/drill/pull/2813#issuecomment-1655539951

   Now tests for this PR fail for other reason (likely due to changes in the PR):
   ```
   Error:  Failures: 
   Error:    TestResultSetLoaderOmittedValues.testOmittedValuesAtEndWithOverflow:264 Row 0 col d should be null
   Error:    TestResultSetLoaderOverflow.testBatchSizeLimit:164 expected:<16385> but was:<8193>
   Error:    TestOutputBatchSize.testSizerRepeatedRepeatedList:2922 expected:<1048576> but was:<1048560>
   Error:  Errors: 
   Error:    TestResultSetLoaderOmittedValues>SubOperatorTest.classTeardown:39 ยป IllegalState Allocator[ROOT] closed with outstanding buffers allocated (9).
   ```


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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #2813:
URL: https://github.com/apache/drill/pull/2813#issuecomment-1605613862

   Some tests failing with this error - `IO Running in secure mode, but config doesn't have a keytab`
   
   Seems like a Hadoop issue. We may need to upgrade Hadoop or at least review the Hadoop config used in the 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.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "cgivre (via GitHub)" <gi...@apache.org>.
cgivre commented on PR #2813:
URL: https://github.com/apache/drill/pull/2813#issuecomment-1605819647

   @pjfanning I think we may have a CI issue.  @vvysotskyi @jnturton Any ideas here?  Could this be related to that issue we encountered before with the CI and connections?


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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning closed pull request #2813: DRILL-8443: Upgrade netty due to CVE
URL: https://github.com/apache/drill/pull/2813


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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "jnturton (via GitHub)" <gi...@apache.org>.
jnturton commented on PR #2813:
URL: https://github.com/apache/drill/pull/2813#issuecomment-1872676811

   @pjfanning @cgivre, @rymarm has made progress on this in #2857. Shall we close this one and try to get that one over the line?


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

Re: [PR] DRILL-8443: Upgrade netty due to CVE (drill)

Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #2813:
URL: https://github.com/apache/drill/pull/2813#issuecomment-1872888170

   sure - I haven't been looking at 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