Skip to content

[cherry-pick] Fix install.sh for DS 2.0 and add comment to install_config.conf (#6506)#6525

Merged
lenboo merged 1 commit intoapache:2.0-preparefrom
zhongjiajie:cp-6506
Oct 14, 2021
Merged

[cherry-pick] Fix install.sh for DS 2.0 and add comment to install_config.conf (#6506)#6525
lenboo merged 1 commit intoapache:2.0-preparefrom
zhongjiajie:cp-6506

Conversation

@zhongjiajie
Copy link
Member

  • Fix install.sh for DS 2.0

  • Use #* make it work in macos

  • Add comment for properties change

  • Add JAVA HOME for only one file change to deploy

  • Reduce config change by using default value of: plugin registry, alert, task as

  • Change javaHome default value

  • Delete config param zkQuorum

  • Reorder param

  • Add comment for install_config.conf

…che#6506)

* Fix install.sh for DS 2.0

* Use #* make it work in macos

* Add comment for properties change

* Add JAVA HOME for only one file change to deploy

* Reduce config change by using default value of: plugin registry, alert, task as

* Change javaHome default value

* Delete config param zkQuorum

* Reorder param

* Add comment for install_config.conf
@zhongjiajie zhongjiajie changed the title Fix install.sh for DS 2.0 and add comment to install_config.conf (#6506) [cherry-pick] Fix install.sh for DS 2.0 and add comment to install_config.conf (#6506) Oct 14, 2021
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

29.4% 29.4% Coverage
0.0% 0.0% Duplication

@zhongjiajie
Copy link
Member Author

PTAL @CalvinKirs @lenboo thanks

Copy link
Contributor

@lenboo lenboo left a comment

Choose a reason for hiding this comment

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

+1

@lenboo lenboo merged commit 5f19d5b into apache:2.0-prepare Oct 14, 2021
@zhongjiajie zhongjiajie deleted the cp-6506 branch October 14, 2021 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants