hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28163: Upgrade apache directory server to 2.0.0-M24

Open Aggarwal-Raghav opened this issue 1 year ago • 4 comments

What changes were proposed in this pull request?

Please refer to HIVE-28163 for stacktrace

Why are the changes needed?

When running Hive3 + (patches from master branch), with jdk17, there were failure in service module regarding apache directory server. Here is the test which failed. mvn test -Dtest='org.apache.hive.service.auth.TestLdapAtnProviderWithMiniDS#testGroupFilterPositive' -pl service

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

Yes. Attaching updated dependency tree: new-dependency-tree.txt

How was this patch tested?

In my org we have hive3 + master branch patches, with jdk17. There we are running it. Will see the UT once by running it here.

Aggarwal-Raghav avatar Mar 29 '24 08:03 Aggarwal-Raghav

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Mar 29 '24 18:03 sonarqubecloud[bot]

@ayushtkn, can you please review this.

Aggarwal-Raghav avatar Apr 26 '24 11:04 Aggarwal-Raghav

There is a JIRA DIRSERVER-2386 which was raised as part of https://github.com/apache/hive/pull/5044, Until that is resolved, I think moving to 2.0.0-M24 is better because we are using very old version of apache directory server.

Aggarwal-Raghav avatar Apr 27 '24 16:04 Aggarwal-Raghav

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.

github-actions[bot] avatar Jun 27 '24 00:06 github-actions[bot]

@Aggarwal-Raghav can you reopen this PR, if the build is clean I will get it merged

ayushtkn avatar Apr 02 '25 08:04 ayushtkn

@Aggarwal-Raghav can you reopen this PR, if the build is clean I will get it merged

Sure @ayushtkn , let me recreate the PR. Thanks

Aggarwal-Raghav avatar Apr 02 '25 08:04 Aggarwal-Raghav