[ 
https://issues.apache.org/jira/browse/HIVE-10674?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sushanth Sowmyan updated HIVE-10674:
------------------------------------
    Description: 
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.


  was:
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 : 



> HIVE-9302 introduces 2 jars in 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
>            Assignee: 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)

Reply via email to