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

Update the aws bootstrap script with correct version #2980

Merged
merged 4 commits into from
Apr 12, 2023

Conversation

ericsun95
Copy link
Contributor

Update the bootstrap script with correct scala version and hbase version. Since we drop the support for 2.11 and we are moving towards hbase 2, I just change it to two by default.

Update the bootstrap script with correct scala version and hbase version. Since we drop the support for 2.11 and we are moving towards hbase 2, I just change it to two by default.
Update the bootstrap script with correct version
@ericsun95 ericsun95 changed the title Update the bootstrap script with correct version Update the aws bootstrap script with correct version Apr 6, 2023
@ericsun95
Copy link
Contributor Author

@ronq @jrs53 @elahrvivaz first to make change to this repo, feel free to comment and review! Thanks

@elahrvivaz
Copy link
Contributor

thanks for fixing this!

ericsun95 and others added 2 commits April 10, 2023 18:51
…aws.sh

Co-authored-by: Emilio <elahrvivaz@ccri.com>
…aws.sh

Co-authored-by: Emilio <elahrvivaz@ccri.com>
@ericsun95
Copy link
Contributor Author

thanks for fixing this!

You are welcome. Committed the suggested changes!

@elahrvivaz
Copy link
Contributor

Thanks again! I verified that the vars are being replaced as expected, so looks good to me. I think even though this is a small contribution, before we can merge it you will have to create an Eclipse account and sign the contributor license agreement, as linked here. Sorry for the hassle!

@ericsun95
Copy link
Contributor Author

Thanks again! I verified that the vars are being replaced as expected, so looks good to me. I think even though this is a small contribution, before we can merge it you will have to create an Eclipse account and sign the contributor license agreement, as linked here. Sorry for the hassle!

Hey, done!

@elahrvivaz elahrvivaz merged commit 5a4656d into locationtech:main Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants