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

ASF GitHub Bot commented on FLINK-8193:
---------------------------------------

Github user zentol commented on the issue:

    https://github.com/apache/flink/pull/5118
  
    that's what we're already doing (well technicall yin the `pre-package` 
phase).
    
    Thing is that the `package` phase is _always_ executed before `verify`, and 
we can't get around that aside from building a custom life-cycle. And we have 
to print a warning in this case, as the jar is still created and thus could be 
used.


> Rework quickstart exclusions
> ----------------------------
>
>                 Key: FLINK-8193
>                 URL: https://issues.apache.org/jira/browse/FLINK-8193
>             Project: Flink
>          Issue Type: Improvement
>          Components: Build System, Quickstarts
>    Affects Versions: 1.4.0
>            Reporter: Chesnay Schepler
>            Assignee: Chesnay Schepler
>
> The quickstart poms currently contain 2 entirely separate dependency 
> exclusion mechanisms:
> A) manual exclusions of dependencies (including transitive dependencies) via 
> the {{artifactSet}} of the {{maven-shade-plugin}}
> B) automatic inclusion by marking dependencies as {{<provided>}} with the 
> {{build-jar}} profile
> I propose removing A) entirely. By default, all dependencies will be included 
> in the jar to allow execution in the IDE (which setting all dependencies to 
> provided would not). To execute things on a cluster we now always refer to 
> Option B), which is trivial to maintain and also reliable.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to