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

Update scope on a property #11111

Merged
merged 1 commit into from Mar 3, 2020
Merged

Update scope on a property #11111

merged 1 commit into from Mar 3, 2020

Conversation

apc999
Copy link
Contributor

@apc999 apc999 commented Mar 3, 2020

Update scope

@apc999 apc999 requested a review from LuQQiu March 3, 2020 21:40
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • AmplabJenkins build check: PENDING
    • We were not able to detect AmplabJenkins test results on this PR. Status will update when testing completes.
  • Commits associated with Github account: PASS
  • PR title follows the conventions: PASS

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

LuQQiu
LuQQiu approved these changes Mar 3, 2020
View reviewed changes
Copy link
Contributor

@LuQQiu LuQQiu left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@AmplabJenkins
Copy link

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8659/
Test PASSed.

@alluxio-bot
Copy link
Contributor

Automated checks report:

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

All checks passed!

@alluxio-bot alluxio-bot added the API Change Changes covering public API label Mar 3, 2020
@apc999
Copy link
Contributor Author

apc999 commented Mar 3, 2020

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 42dbb19 into master Mar 3, 2020
@apc999 apc999 deleted the apc999-patch-1 branch March 17, 2020 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LuQQiu LuQQiu LuQQiu approved these changes

Assignees

@gpang gpang

Labels
API Change Changes covering public API
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@apc999 @alluxio-bot @AmplabJenkins @LuQQiu @gpang