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.

Sign up for GitHub

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

Jump to bottom

alluxio-start.sh masters failure #15795

Open
JySongWithZhangCe opened this issue Jun 29, 2022 · 5 comments
Open

alluxio-start.sh masters failure #15795

JySongWithZhangCe opened this issue Jun 29, 2022 · 5 comments
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-bug This issue is about a bug

Comments

@JySongWithZhangCe
Copy link
Contributor

JySongWithZhangCe commented Jun 29, 2022 •
edited
Loading

Alluxio Version:
2.7.4

Describe the bug
When there are some WARN info of ALLUXIO_MASTER_JAVA_OPTS, alluxio-start.sh masters failure.

Urgency
Urgent.

Are you planning to fix it
Romove ALLUXIO_MASTER_JAVA_OPTS in https://github.com/Alluxio/alluxio/blob/master/bin/alluxio-masters.sh line 41 & 42, and in https://github.com/Alluxio/alluxio/blob/master/bin/alluxio-monitor.sh line 194 & 195.

Additional context
WARN info makes HA_ENABLED=$(${BIN}/alluxio getConf ${ALLUXIO_MASTER_JAVA_OPTS} alluxio.zookeeper.enabled) != true
@JySongWithZhangCe JySongWithZhangCe added the type-bug This issue is about a bug label Jun 29, 2022
@JySongWithZhangCe
Copy link
Contributor Author

@beinan Would you like to take a look~Thanks♪(･ω･)ﾉ

@boobpoop
Copy link

properties of alluxio.zookeeper.enabled and alluxio.master.journal.type are used to do master election, it's not a good idea to remove it.

@JySongWithZhangCe
Copy link
Contributor Author

JySongWithZhangCe commented Jun 29, 2022 •
edited
Loading

properties of alluxio.zookeeper.enabled and alluxio.master.journal.type are used to do master election, it's not a good idea to remove it.

${ALLUXIO_MASTER_JAVA_OPTS} seems not necessary.

@JySongWithZhangCe
Copy link
Contributor Author

@jiacheliu3 Do you have any advice? Thanks♪(･ω･)ﾉ

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The PR/Issue does not have recent activities and will be closed automatically label Jan 31, 2023
This was referenced Oct 26, 2023
Closed
Merged
alluxio-bot pushed a commit that referenced this issue Nov 6, 2023
@YichuanSun
Fix getConf bugs when it comes across warning
66ca4b6 
### What changes are proposed in this pull request?

As issue #15795 mentioned. When output any warnings, the cli of judging HA_ENABLED will be invalid. This PR fixs it.

### Why are the changes needed?

Bug fix.

### Does this PR introduce any user facing changes?

No.

			pr-link: #18334
			change-id: cid-c0d595d263788c5e64e6ce4508e2ee120b527b74
maobaolong pushed a commit to maobaolong/alluxio that referenced this issue Jan 3, 2024
@YichuanSun @codings-dan
Fix getConf bugs when it comes across warning
da89265 
### What changes are proposed in this pull request?

As issue Alluxio#15795 mentioned. When output any warnings, the cli of judging HA_ENABLED will be invalid. This PR fixs it.

### Why are the changes needed?

Bug fix.

### Does this PR introduce any user facing changes?

No.

			pr-link: Alluxio#18334
			change-id: cid-c0d595d263788c5e64e6ce4508e2ee120b527b74
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-bug This issue is about a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JySongWithZhangCe @boobpoop