Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix local gradle build #551

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

Xtansia
Copy link
Collaborator

@Xtansia Xtansia commented Feb 11, 2025

Description

The default versions of hadoop/spark/hive used in some integration testing were very old and are no longer present in maven, this updates the versions so that running ./gradlew build now passes locally.

Relates to this slack thread: https://opensearch.slack.com/archives/C051XNH62KB/p1736806808963089
Screenshot 2025-02-12 at 10 22 00 AM

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Xtansia Xtansia force-pushed the fix-local-gradle-build branch 2 times, most recently from afc5980 to 736fbac Compare February 11, 2025 21:50
Copy link
Collaborator

@harshavamsi harshavamsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thnx for the changes @Xtansia!

@@ -1,4 +1,4 @@
opensearch_hadoop = 1.3.0
opensearch = 2.13.0
opensearch = 2.18.0
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be changed to 2.19.0, OS just released

Signed-off-by: Thomas Farr <tsfarr@amazon.com>
@Xtansia Xtansia force-pushed the fix-local-gradle-build branch from 736fbac to 7b5604c Compare February 12, 2025 21:54
@Xtansia Xtansia merged commit 9224a9c into opensearch-project:main Feb 12, 2025
14 checks passed
@Xtansia Xtansia deleted the fix-local-gradle-build branch February 12, 2025 22:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants