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.

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

HIVE-28228: Remove redundant/unused sources maven profile #5224

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zabetak
Copy link
Contributor

@zabetak zabetak commented Apr 29, 2024

Why are the changes needed?

Simplify build and reduce maintenance overhead

The sources maven profile was introduced in HIVE-5717 among others to package source code into a .jar and publish it to maven. However, after adding Apache pom as parent in HIVE-6608 the sources profile is now redundant; it is subsumed by the apache-release profile.

In some cases, the sources profile allows to package the test sources in a jar file (HIVE-13206) but this profile is not used during the release so the generated test-sources.jars are not consumed by anyone. This seems like an unused dev only feature thus it can be removed.

Does this PR introduce any user-facing change?

Devs will no longer have the option to generate test-sources.jar using the sources profile.

Is the change a dependency upgrade?

No

How was this patch tested?

N/A

The sources maven profile was introduced in HIVE-5717 among others to package source code into a .jar and publish it to maven. However, after adding Apache pom as parent in HIVE-6608 the sources profile is now redundant; it is subsumed by the apache-release profile.

In some cases, the sources profile allows to package the test sources in a jar file (HIVE-13206) but this profile is not used during the release so the generated test-sources.jars are not consumed by anyone. This seems like an unused dev only feature thus it can be removed.
Copy link

sonarcloud bot commented Apr 29, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ayushtkn
Copy link
Member

there is a use case mentioned:
https://issues.apache.org/jira/browse/HIVE-13206?focusedCommentId=15183873&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-15183873

that will get messed up, we do release the src tar, so somebody can build and get the test jar for their use case

@zabetak
Copy link
Contributor Author

zabetak commented Apr 30, 2024

@ayushtkn I consider the use-case in HIVE-13206 as a "dev only" feature and not a very useful one. On the other hand this "feature" adds confusion and maintenance overhead (see exchanges in HIVE-28024).

@zabetak
Copy link
Contributor Author

zabetak commented May 16, 2024

Hey @ayushtkn , did you see my latest comment? If you feel that this profile is useful let me know and I will close the JIRA and PR.

@ayushtkn
Copy link
Member

hey @zabetak
no strong opinions here, if others are convinced, I don't pose any objections

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants