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

alluxio cli needs to show commit id #9930

Closed
cheyang opened this issue Sep 21, 2019 · 0 comments · Fixed by #9936
Closed

alluxio cli needs to show commit id #9930

cheyang opened this issue Sep 21, 2019 · 0 comments · Fixed by #9936
Labels
type-feature This issue is a feature request

Comments

@cheyang
Copy link
Contributor

cheyang commented Sep 21, 2019 •
edited

Is your feature request related to a problem? Please describe.
2.1.0-snapshot
Describe the solution you'd like
when executing alluxio version, it can show commit id. It's helpful to investigate the issues when using the development version.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Urgency
Explain why the feature is important

Additional context
Add any other context or screenshots about the feature request here.
@cheyang cheyang added the type-feature This issue is a feature request label Sep 21, 2019
@cheyang cheyang changed the title alluxio cli should show commit id alluxio cli needs to show commit id Sep 21, 2019
@ZacBlanco ZacBlanco mentioned this issue Sep 22, 2019
Merged
alluxio-bot pushed a commit that referenced this issue Sep 24, 2019
@ZacBlanco @alluxio-bot
Add git revision to ./bin/alluxio version
2e67f3b 
In light of preserving behavior, the command behaves the same as in
previous versions. This change adds the -r option which can be specified
optionally so that users can see part or all of the revision output
appended to the end of the normal version string.

Added two new maven plugins to retrieve the git revision.

Closes #9930

pr-link: #9936
change-id: cid-b4a1836b31b57881a1f9ae675dd53fe4d7534270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-feature This issue is a feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add git revision to ./bin/alluxio version ZacBlanco/alluxio
1 participant
@cheyang