-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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.
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
Fix typo in MetaMasterConfigurationServiceHandler #13737
Conversation
Hi @ns7381, 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. |
You did it @ns7381! Thank you for signing the Contribution License Agreement. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this improvement. it LGTM. @apc999 Please take another look.
Codecov Report
@@ Coverage Diff @@
## master #13737 +/- ##
============================================
- Coverage 43.72% 34.64% -9.08%
+ Complexity 9074 6960 -2114
============================================
Files 1384 1384
Lines 80313 80313
Branches 9768 9768
============================================
- Hits 35114 27827 -7287
- Misses 42269 50091 +7822
+ Partials 2930 2395 -535
Continue to review full report at Codecov.
|
alluxio-bot, merge this please |
Thanks @ns7381 for fixing the typo! |
Fixes Alluxio/Community#605 In https://github.com/Alluxio/alluxio/blob/fb486603b981ef6c04c111799f9ef72630a5eb00/core/server/master/src/main/java/alluxio/master/meta/MetaMasterConfigurationServiceHandler.java#L88 Correct the spell of resquest to be request pr-link: Alluxio#13737 change-id: cid-0afeeecca66576d41ca63a16a34f79717ad91c4d
Fixes Alluxio/Community#605
In
alluxio/core/server/master/src/main/java/alluxio/master/meta/MetaMasterConfigurationServiceHandler.java
Line 88 in fb48660
Correct the spell of resquest to be request