-
Notifications
You must be signed in to change notification settings - Fork 145
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
[531] xtable poms are missing license details. #533
Conversation
pom.xml
Outdated
|
||
<groupId>org.apache.xtable</groupId> | ||
<artifactId>xtable</artifactId> | ||
<name>xtable</name> | ||
<name>XTable Parent Project</name> |
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.
Why do we need this change ?
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.
It will displays the project name clearly while building the xtable. You can refer Spark code for reference.
[INFO] XTable Parent Project .............................. SUCCESS [ 0.316 s]
[INFO] XTable Api Project ................................. SUCCESS [ 0.032 s]
[INFO] XTable Hudi Support Parent Project ................. SUCCESS [ 0.025 s]
[INFO] XTable Hudi Support Utils Project .................. SUCCESS [ 0.019 s]
[INFO] XTable Core Project ................................ SUCCESS [ 0.036 s]
[INFO] XTable Utilities Project ........................... SUCCESS [ 0.014 s]
[INFO] XTable Hudi Support Extensions Project ............. SUCCESS [ 0.015 s]
[INFO] ------------------------------------------------------------------------
@rangareddy Can you update the PR description to |
9ac5da9
to
00301fa
Compare
Hi @vinishjail97 Merge this PR to main branch. |
Important Read
This pull request aims to address issue #531
What is the purpose of the pull request
Adding the tag to xtable project and updating the project name.
Brief change log
Verify this pull request
Verified by running
$ mvn clean install -DskipTests -U
locally.