alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

[DOCFIX] Compile support Apple M1

Open StephenRi opened this issue 3 years ago • 8 comments

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,

  1. If you propose a new API, clarify the use case for a new API.
  2. 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

  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

StephenRi avatar Sep 26 '21 02:09 StephenRi

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.

alluxio-bot avatar Sep 26 '21 02:09 alluxio-bot

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

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

alluxio-bot avatar Sep 26 '21 02:09 alluxio-bot

Automated checks report:

  • Commits associated with Github account: PASS
  • PR title follows the conventions: PASS

All checks passed!

alluxio-bot avatar Sep 26 '21 02:09 alluxio-bot

@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 avatar Sep 27 '21 19:09 yuzhu

@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 avatar Mar 17 '22 06:03 Sucran

@Sucran thanks for the note. Please make a similar PR targeting master branch. We can only take contributions targetting master branch.

yuzhu avatar Mar 17 '22 11:03 yuzhu

@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.

Sucran avatar Mar 31 '22 13:03 Sucran

@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.

Sucran avatar Mar 31 '22 13:03 Sucran

@beinan could you take a look at this? since you have some experience with M1?

yuzhu avatar Nov 03 '22 02:11 yuzhu

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.

github-actions[bot] avatar Jan 27 '23 15:01 github-actions[bot]

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.

github-actions[bot] avatar Jun 15 '23 15:06 github-actions[bot]