Skip to content

Conversation

@JoshRosen
Copy link
Contributor

@JoshRosen JoshRosen commented Sep 20, 2016

This patch updates the kinesis-asl-assembly build to prevent that module from being published as part of Maven releases and snapshot builds.

The kinesis-asl-assembly includes classes from the Kinesis Client Library (KCL) and Kinesis Producer Library (KPL), both of which are licensed under the Amazon Software License and are therefore prohibited from being distributed in Apache releases.

</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-install-plugin</artifactId>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might be ok? (It's supposed to be local only.) But probably good for consistency anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to do this in order to prevent this artifact from being published to a release staging repository, since the current release publishing scripts work by first installing the components to be published into a fresh temporary Maven repository and then searching that repository for artifacts, signing them, and uploading them to the staging repository using curl. See

$MVN -DzincPort=$ZINC_PORT -Dmaven.repo.local=$tmp_repo -DskipTests $PUBLISH_PROFILES clean install

@SparkQA
Copy link

SparkQA commented Sep 20, 2016

Test build #65681 has finished for PR 15167 at commit 3220800.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Sep 21, 2016

Oh right, that's a nicer solution. LGTM.

@JoshRosen
Copy link
Contributor Author

I'm going to merge this into master, branch-2.0, and branch-1.6.

asfgit pushed a commit that referenced this pull request Sep 21, 2016
…shed

This patch updates the `kinesis-asl-assembly` build to prevent that module from being published as part of Maven releases and snapshot builds.

The `kinesis-asl-assembly` includes classes from the Kinesis Client Library (KCL) and Kinesis Producer Library (KPL), both of which are licensed under the Amazon Software License and are therefore prohibited from being distributed in Apache releases.

Author: Josh Rosen <[email protected]>

Closes #15167 from JoshRosen/stop-publishing-kinesis-assembly.

(cherry picked from commit d7ee122)
Signed-off-by: Josh Rosen <[email protected]>
@asfgit asfgit closed this in d7ee122 Sep 21, 2016
@JoshRosen JoshRosen deleted the stop-publishing-kinesis-assembly branch September 21, 2016 18:43
asfgit pushed a commit that referenced this pull request Sep 21, 2016
…shed

This patch updates the `kinesis-asl-assembly` build to prevent that module from being published as part of Maven releases and snapshot builds.

The `kinesis-asl-assembly` includes classes from the Kinesis Client Library (KCL) and Kinesis Producer Library (KPL), both of which are licensed under the Amazon Software License and are therefore prohibited from being distributed in Apache releases.

Author: Josh Rosen <[email protected]>

Closes #15167 from JoshRosen/stop-publishing-kinesis-assembly.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Sep 22, 2016
…shed

This patch updates the `kinesis-asl-assembly` build to prevent that module from being published as part of Maven releases and snapshot builds.

The `kinesis-asl-assembly` includes classes from the Kinesis Client Library (KCL) and Kinesis Producer Library (KPL), both of which are licensed under the Amazon Software License and are therefore prohibited from being distributed in Apache releases.

Author: Josh Rosen <[email protected]>

Closes apache#15167 from JoshRosen/stop-publishing-kinesis-assembly.

(cherry picked from commit ce0a222)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants