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

SAMZA-2708: Update yarn from 2.7.1 to 2.10.1 #1557

Merged
merged 3 commits into from
Nov 11, 2021
Merged

Conversation

APB0001
Copy link
Contributor

@APB0001 APB0001 commented Nov 10, 2021

Issues:
The version of YARN used by the Samza project is outdated and needs to be updated.
https://issues.apache.org/jira/browse/SAMZA-2708

Changes:
1. Update yarn from 2.7.1 to 2.10.1. Other minor fixes to get this to work.
2. Use https rather than http to avoid a build hang when the build script references "repo.typesafe.com/typesafe/releases".
3. Update references of "samza-test_2.11-1.6.0-SNAPSHOT.tgz" to "samza-test_2.11-1.7.0-SNAPSHOT.tgz" to be consistent with the file actually produced by the integration tests.

Tests:
'build test' and run yarn integration tests

API Changes: None
Upgrade Instructions: None
Usage Instructions: None

Tony Brusseau added 3 commits November 9, 2021 18:02
…est_2.11-1.7.0-SNAPSHOT.tgz" to be consistent with the file actually produced by the integration tests.
…rences "repo.typesafe.com/typesafe/releases".
…rds compatablity issues exposed by this change.
Copy link
Contributor

@bringhurst bringhurst left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@Sanil15 Sanil15 left a comment

Choose a reason for hiding this comment

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

Thanks for the upgrade

@Sanil15
Copy link
Contributor

Sanil15 commented Nov 11, 2021

Change looks good, minor thing can you please follow the following wiki: https://cwiki.apache.org/confluence/display/SAMZA/SEP-25%3A+PR+Title+And+Description+Guidelines

for PR title & description format

@APB0001 APB0001 changed the title Update yarn from 2.7.1 to 2.10.1. SAMZA-2708: Update yarn from 2.7.1 to 2.10.1 Nov 11, 2021
@APB0001
Copy link
Contributor Author

APB0001 commented Nov 11, 2021

Change looks good, minor thing can you please follow the following wiki: https://cwiki.apache.org/confluence/display/SAMZA/SEP-25%3A+PR+Title+And+Description+Guidelines

for PR title & description format

Updated.

@Sanil15 Sanil15 merged commit a6ebddd into apache:master Nov 11, 2021
ehoner pushed a commit to ehoner/samza that referenced this pull request Apr 11, 2023
The version of YARN used by the Samza project is outdated and needs to be updated.

Co-authored-by: Tony Brusseau <tbrussea@tbrussea-mn1.linkedin.biz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants