[jira] [Commented] (BEAM-450) Modules are shaded to the same path

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEAM-450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15752612#comment-15752612 ] 

ASF GitHub Bot commented on BEAM-450:
-------------------------------------

GitHub user dhalperi opened a pull request:

    https://github.com/apache/incubator-beam/pull/1633

    [BEAM-450] Shade modules to separate paths

    R: @lukecwik 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dhalperi/incubator-beam shading-package-names

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-beam/pull/1633.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1633
    
----
commit 0bed1b3c494ec23f3ca18389d7b3bce15e0bd363
Author: Dan Halperin <dh...@google.com>
Date:   2016-12-15T21:50:39Z

    [BEAM-450] Shade modules to separate paths

----


> Modules are shaded to the same path
> -----------------------------------
>
>                 Key: BEAM-450
>                 URL: https://issues.apache.org/jira/browse/BEAM-450
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>    Affects Versions: 0.1.0-incubating, 0.2.0-incubating
>            Reporter: Daniel Halperin
>            Assignee: Daniel Halperin
>              Labels: newbie, starter
>
> Right now multiple modules are using the same repackaged path. We should be using per-artifact paths so that they don't conflict.
> One proposal was simply to adopt {{${project.groupId}.${project.artifactId}.repackaged}} as the shading location. If it works.
> This is a good starter issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)