Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRILL-8443: Upgrade netty due to CVE #2813

Closed
wants to merge 2 commits into from
Closed

DRILL-8443: Upgrade netty due to CVE #2813

wants to merge 2 commits into from

Conversation

pjfanning
Copy link
Contributor

@pjfanning pjfanning changed the title [DRILL-8443] Upgrade netty due to CVE DRILL-8443: Upgrade netty due to CVE Jun 24, 2023
@pjfanning
Copy link
Contributor Author

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.

@cgivre
Copy link
Contributor

cgivre commented Jun 25, 2023

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

@cgivre cgivre self-assigned this Jun 25, 2023
@vvysotskyi
Copy link
Member

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

@jnturton
Copy link
Contributor

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

@pjfanning
Copy link
Contributor Author

sure - I haven't been looking at this

@pjfanning pjfanning closed this Dec 31, 2023
@pjfanning pjfanning deleted the netty-upgrade branch December 31, 2023 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants