[
https://issues.apache.org/jira/browse/HIVE-10674?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yu Wei updated HIVE-10674:
--------------------------
Assignee: (was: Yu Wei)
> jars should not be checked in to the source control repo
> --------------------------------------------------------
>
> Key: HIVE-10674
> URL: https://issues.apache.org/jira/browse/HIVE-10674
> Project: Hive
> Issue Type: Bug
> Affects Versions: 1.2.0, 1.3.0
> Reporter: Sushanth Sowmyan
> Priority: Blocker
>
> The 2 jars added by HIVE-9302 run afoul of the source package generation as
> part of the 1.2 release, since a source package is not supposed to contain
> any binaries. If we have binaries, they're supposed to be brought in as a
> download step during the compile or test-compile phase from a well-known
> published location such as a maven repository. The postgres jar we can depend
> on as a download, and it is an open source product that is compatible with
> the Apache License, but DummyDriver is worse, because there is no source
> attached to it either, which makes it not okay to include in the binary
> release of hive either.
> Thus, for branch-1.2, I am going to do a git rm of those two jars right away.
> This, unfortunately, might cause a few tests added here to fail for
> branch-1.2, but this should be acceptable for the time being.
> I'm opening this jira to track the following:
> a) git rm of the postgres and DummyDriver jar from master
> b) adding source code for DummyDriver into master, and changing the build so
> we depend on it being compiled, rather than included from test-resources.
> c) changing the postgres inclusion to a download.
> This should also be applied to branch-1.2 after release, preferably before
> 1.2.1, so that future updates of 1.2 have this fixed as well.
> Note: In addition, there are two more jars added by HIVE-7553 which also need
> to have this same fix applied to them :
> ql/src/test/resources/SessionStateTest.jar.v1 and
> ql/src/test/resources/SessionStateTest.jar.v1 . These are included in a
> similar manner, for tests, but are binaries without source code attached.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
