alluxio
alluxio copied to clipboard
[DOCFIX] Compile support Apple M1
What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,
- If you propose a new API, clarify the use case for a new API.
- If you fix a bug, describe the bug.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including
- change in user-facing APIs
- addition or removal of property keys
- webui
Hi @StephenRi, thanks for your contribution!
In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.
Automated checks report:
- Commits associated with Github account: PASS
- PR title follows the conventions: FAIL
- The title of the PR does not pass all the checks. Please fix the following issues:
- First word must be capitalized
- The title of the PR does not pass all the checks. Please fix the following issues:
Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.
Automated checks report:
- Commits associated with Github account: PASS
- PR title follows the conventions: PASS
All checks passed!
@StephenRi thanks for improving our docs. Do you mind adding a similar section to the english docs as well? and please target the master branch with your doc change. We will work on cherrypicking it to the other branches.
@yuzhu Hi, could you please review this and merge into the release_branch? I'm working with apple m1 silicon and build the source code with the steps of Alluxio Documents. But I met two issue, one was solved by this PR, another needs to upgrade the version of plugin "frontend-maven-plugin" from 1.6 to 1.11.0. (ref to the link )Otherwise alluxio-webui can not download the local nodejs and npm version. Hope this would help. If you need my help, just @ me.
@Sucran thanks for the note. Please make a similar PR targeting master branch. We can only take contributions targetting master branch.
@yuzhu Could you please merge this change into master branch first, so I can edit on the change of @StephenRi, I found the RocksDB JNI version needs to upgrade to 6.29.x, otherwise the Alluxio-master can not start.
@yuzhu Could you please merge this change into master branch first, so I can edit on the change of @StephenRi, I found the RocksDB JNI version needs to upgrade to 6.29.x, otherwise the Alluxio-master can not start.
if you need me to take this PR's context into my PR together, then I will happy to do this. But I'm just confused now.
@beinan could you take a look at this? since you have some experience with M1?
This pull request 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.
This pull request 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.