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

[SMALLFIX] Java 8 migration: replace anonymous type with lambda in alluxio.web.UIFileInfo#PATH_STRING_COMPARE #8002

Merged
merged 9 commits into from
Nov 5, 2018
Merged

[SMALLFIX] Java 8 migration: replace anonymous type with lambda in alluxio.web.UIFileInfo#PATH_STRING_COMPARE #8002

merged 9 commits into from
Nov 5, 2018

Conversation

MoxinC
Copy link
Contributor

@MoxinC MoxinC commented Oct 18, 2018

No description provided.

@alluxio-bot
Copy link
Contributor

Hi @MoxinC, 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.

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 18, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 18, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC MoxinC mentioned this pull request Oct 18, 2018
Closed
@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 19, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 19, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 19, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 19, 2018

Jenkins, test this please

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

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

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 19, 2018

Jenkins, test this please

@alluxio-bot
Copy link
Contributor

You did it @MoxinC!

Thank you for signing the Contribution License Agreement.

@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/740/
Test PASSed.

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 22, 2018

MoxinC wants to merge into Alluxio:master from MoxinC:smallfix

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 23, 2018

@apc999

@MoxinC
Copy link
Contributor Author

MoxinC commented Oct 24, 2018

@apc999
MoxinC wants to merge into Alluxio:master from MoxinC:smallfix

@newnius
Copy link
Contributor

newnius commented Oct 28, 2018

@apc999 PTAL, thanks

@newnius
Copy link
Contributor

newnius commented Nov 2, 2018

LGTM

@apc999 apc999 merged commit 60c1be8 into Alluxio:master Nov 5, 2018
@MoxinC MoxinC mentioned this pull request Nov 5, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apc999 apc999 apc999 approved these changes

Assignees

@apc999 apc999

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@MoxinC @alluxio-bot @AmplabJenkins @newnius @apc999