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

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

GitHub user bowenli86 opened a pull request:

    https://github.com/apache/flink/pull/5131

    [FLINK-8218] move flink-connector-kinesis examples from /src to /test

    ## What is the purpose of the change
    
    just saw there are runnable examples in src tree and traced back to this 
PR. I feel it's really bad to have runnable examples (with java main method) in 
a lib jar that Flink distributes.
    
    chatted with Gordon, we agreed that we should move those examples from /src 
to /test
    
    ## Brief change log
    
    move examples in flink-connector-kinesis from /src to /test
    
    ## Verifying this change
    
    This change is a trivial rework / code cleanup without any test coverage.
    
    ## Does this pull request potentially affect one of the following parts:
    
    none
    
    ## Documentation
    
    none

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

    $ git pull https://github.com/bowenli86/flink FLINK-8218

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

    https://github.com/apache/flink/pull/5131.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 #5131
    
----

----


> move flink-connector-kinesis examples from /src to /test
> --------------------------------------------------------
>
>                 Key: FLINK-8218
>                 URL: https://issues.apache.org/jira/browse/FLINK-8218
>             Project: Flink
>          Issue Type: Improvement
>          Components: Kinesis Connector
>    Affects Versions: 1.5.0
>            Reporter: Bowen Li
>            Assignee: Bowen Li
>            Priority: Minor
>             Fix For: 1.5.0
>
>
> I just saw there are runnable examples in src tree and traced back to this 
> PR. I feel it's really bad to have runnable examples (with java main method) 
> in a lib jar that Flink distributes.
> chatted with [~tzulitai] , we agreed that we should move those examples from 
> /src to /test



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

Reply via email to