Mailing-List: contact dev-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CANC1h_vBTA4UEHaB=CupWjRxeGOfzknnOX6BHhSaVUJBHc1aMQ@mail.gmail.com>
References: 
 <CANC1h_u6qtEsF1WCcoU1d38JGd+XTAQWmvp9Stx4vfe68BOjBw@mail.gmail.com>
 <CALuGr6aBs90brHwHSDbjN7k9RaaTryUKQm_8ADtTO8xc=cOiEw@mail.gmail.com>
 <560DB0E8.4020602@apache.org>
 <CAGco--YPADxHF0TyjQHYxe9UjdrYg+4pR9LF+cNxwi+4ikm4hg@mail.gmail.com>
 <CAC27z=PDxAGo7PVqvEH-GC98yrdUdaFdS6C8qfUiRLG0i=i3_Q@mail.gmail.com>
 <CANC1h_v1LtyqtPdTHPYp3XEbN4KprvO39i8d0-SpDM8q9Q_gZw@mail.gmail.com>
 <560E440F.6080307@apache.org> <560E4C9D.2070408@apache.org>
 <CALuGr6Zb0nySmBQXK24fvzVBZ9pf_Li8Fv0t2rHR30i=C7_LvA@mail.gmail.com>
 <560ec5c5.e127b40a.d1d8e.0bc0@mx.google.com>
 <CAGr9p8BFH0q_ZS7Cp=ASgMNyp4oZCDMLpj4GVzSOHPk7fzpOjA@mail.gmail.com>
 <CAAdrtT27hm1MAQDb1cmeR7qCnndivm1YanZOdxabWhaUWWxnyQ@mail.gmail.com>
 <CAELUF_CvK=t1sbO_zvBjBmXyNH_bVnqooH3d2Ovhy+AE-PtPZQ@mail.gmail.com>
 <CANC1h_vBTA4UEHaB=CupWjRxeGOfzknnOX6BHhSaVUJBHc1aMQ@mail.gmail.com>
From: Flavio Pompermaier <pompermaier@okkam.it>
Date: Mon, 11 Jan 2016 14:05:31 +0100
Message-ID: 
 <CAELUF_DaJCcY0JQJNMHq84FbPDGkJs0B9C5hkvBJ=56Dc4jeXA@mail.gmail.com>
Subject: Re: Pulling Streaming out of staging and project restructure
To: dev@flink.apache.org
Content-Type: multipart/alternative; boundary=001a11444b90a64fbb05290e9890

--001a11444b90a64fbb05290e9890
Content-Type: text/plain; charset=UTF-8

Yes I know that right nowthat's to way to compile Flink without tests but
from my perspective (as a developer) that's a workaround. If I don't want
to compile the test classes I should be able to skip it (also because in
Flink compiling tests take quite a lot).
Of course it's not a big deal, it's just a matter of following Maven
principle of convention-over-configuration, so test sources should go in
test folders/jars.

On Mon, Jan 11, 2016 at 1:54 PM, Stephan Ewen <sewen@apache.org> wrote:

> Hi Flavio!
>
> You can always do "mvn -DskipTests clean package". That compiles tests, but
> does not execute them.
>
> Stephan
>
>
> On Mon, Jan 11, 2016 at 11:34 AM, Flavio Pompermaier <pompermaier@okkam.it
> >
> wrote:
>
> > I hope that it's not too late to suggest to add to restructuring also
> > https://issues.apache.org/jira/browse/FLINK-1827 so that to be able to
> > compile Flink without compiling also tests (-Dmaven.test.skip=true) and
> > save a lot of time...
> > I should be fairly easy to fix that.
> >
> > Best,
> > Flavio
> >
> > On Wed, Jan 6, 2016 at 4:22 PM, Fabian Hueske <fhueske@gmail.com> wrote:
> >
> > > Stephan already opened a PR for restructuring the examples:
> > >
> > > --> https://github.com/apache/flink/pull/1482
> > >
> > > Otherwise +1!
> > >
> > > 2016-01-06 16:21 GMT+01:00 Robert Metzger <rmetzger@apache.org>:
> > >
> > > > The renaming hasn't been done completely. I would like to fix this
> > before
> > > > the 1.0.0 release.
> > > >
> > > > I think the following issues are still open:
> > > > - Merge flink-java-examples, flink-scala-examples and
> > > > flink-streaming-examples into "flink-examples" + build the streaming
> > > > examples for the binary build as well.
> > > > - move flink-table to flink-libraries
> > > > - move some modules from flink-staging to flink-batch-connectors
> > > >
> > > >
> > > > If nobody disagrees, I'll open a PR for these changes.
> > > > I created a JIRA for this discussion:
> > > > https://issues.apache.org/jira/browse/FLINK-3205
> > > >
> > > >
> > > > On Fri, Oct 2, 2015 at 7:58 PM, fhueske <fhueske@gmail.com> wrote:
> > > >
> > > > > +1
> > > > >
> > > > >
> > > > > From: Henry Saputra
> > > > > Sent: Friday, October 2, 2015 19:34
> > > > > To: dev@flink.apache.org
> > > > > Subject: Re: Pulling Streaming out of staging and project
> restructure
> > > > >
> > > > >
> > > > > +1
> > > > >
> > > > > On Friday, October 2, 2015, Matthias J. Sax <mjsax@apache.org>
> > wrote:
> > > > >
> > > > > > It think, rename "flink-storm-compatibility-core" to just
> > > "flink-storm"
> > > > > > would be the cleanest solution.
> > > > > >
> > > > > > So in flink-contrib there would be two modules:
> > > > > >   - flink-storm
> > > > > >   - flink-storm-examples
> > > > > >
> > > > > > Please let me know if you have any objection about it.
> > > > > >
> > > > > > -Matthias
> > > > > >
> > > > > > On 10/02/2015 10:45 AM, Matthias J. Sax wrote:
> > > > > > > Sure. Will do that.
> > > > > > >
> > > > > > > -Matthias
> > > > > > >
> > > > > > > On 10/02/2015 10:35 AM, Stephan Ewen wrote:
> > > > > > >> @Matthias: How about getting rid of the
> > storm-compatibility-parent
> > > > and
> > > > > > >> making the core and examples projects directly projects in
> > > "contrib"
> > > > > > >>
> > > > > > >> On Fri, Oct 2, 2015 at 10:34 AM, Till Rohrmann <
> > > > trohrmann@apache.org
> > > > > > <javascript:;>> wrote:
> > > > > > >>
> > > > > > >>> +1 for the new project structure. Getting rid of our code
> dump
> > > is a
> > > > > > good
> > > > > > >>> thing.
> > > > > > >>>
> > > > > > >>> On Fri, Oct 2, 2015 at 10:25 AM, Maximilian Michels <
> > > > mxm@apache.org
> > > > > > <javascript:;>>
> > > > > > >>> wrote:
> > > > > > >>>
> > > > > > >>>> +1 Matthias, let's limit the overhead this has for the
> module
> > > > > > >>> maintainers.
> > > > > > >>>>
> > > > > > >>>> On Fri, Oct 2, 2015 at 12:17 AM, Matthias J. Sax <
> > > > mjsax@apache.org
> > > > > > <javascript:;>>
> > > > > > >>> wrote:
> > > > > > >>>>> I will commit something to flink-storm-compatibility
> tomorrow
> > > > that
> > > > > > >>>>> contains some internal package restructuring. I think,
> > renaming
> > > > the
> > > > > > >>>>> three modules in this commit would be a smart move as both
> > > > changes
> > > > > > >>>>> result in merge conflicts when rebasing open PRs. Thus we
> can
> > > > limit
> > > > > > >>> this
> > > > > > >>>>> pain to a single time. If no objections, I will commit
> those
> > > > > changes
> > > > > > >>>>> tomorrow.
> > > > > > >>>>>
> > > > > > >>>>> -Matthias
> > > > > > >>>>>
> > > > > > >>>>> On 10/01/2015 09:52 PM, Henry Saputra wrote:
> > > > > > >>>>>> +1
> > > > > > >>>>>>
> > > > > > >>>>>> I like the idea moving "staging" projects into appropriate
> > > > > modules.
> > > > > > >>>>>>
> > > > > > >>>>>> While we are at it, I would like to propose changing "
> > > > > > >>>>>> flink-hadoop-compatibility" to "flink-hadoop". It is in my
> > > > bucket
> > > > > > list
> > > > > > >>>>>> but would be nice if it is part of re-org.
> > > > > > >>>>>> Supporting Hadoop in the connector implicitly means
> > > > compatibility
> > > > > > with
> > > > > > >>>> Hadoop.
> > > > > > >>>>>> Also same thing with "flink-storm-compatibility" to
> > > > "flink-storm".
> > > > > > >>>>>>
> > > > > > >>>>>> - Henry
> > > > > > >>>>>>
> > > > > > >>>>>> On Thu, Oct 1, 2015 at 3:25 AM, Stephan Ewen <
> > > sewen@apache.org
> > > > > > <javascript:;>>
> > > > > > >>> wrote:
> > > > > > >>>>>>> Hi all!
> > > > > > >>>>>>>
> > > > > > >>>>>>> We are making good headway with reworking the last parts
> of
> > > the
> > > > > > >>> Window
> > > > > > >>>> API.
> > > > > > >>>>>>> After that, the streaming API should be good to be pulled
> > out
> > > > of
> > > > > > >>>> staging.
> > > > > > >>>>>>>
> > > > > > >>>>>>> Since we are reorganizing the projects as part of that, I
> > > would
> > > > > > shift
> > > > > > >>>> a bit
> > > > > > >>>>>>> more to bring things a bit more up to date.
> > > > > > >>>>>>>
> > > > > > >>>>>>> In this restructure, I would like to get rid of the
> > > > > "flink-staging"
> > > > > > >>>>>>> project. Anyone who only uses the maven artifacts sees no
> > > > > > difference
> > > > > > >>>>>>> whether a project is in "staging" or not, so it does not
> > help
> > > > > much
> > > > > > to
> > > > > > >>>> have
> > > > > > >>>>>>> that directory structure.
> > > > > > >>>>>>> On the other hand, projects have a tendency to linger in
> > > > staging
> > > > > > >>>> forever
> > > > > > >>>>>>> (like avro, spargel, hbase, jdbc, ...)
> > > > > > >>>>>>>
> > > > > > >>>>>>> The new structure could be
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-core
> > > > > > >>>>>>> flink-java
> > > > > > >>>>>>> flink-scala
> > > > > > >>>>>>> flink-streaming-core
> > > > > > >>>>>>> flink-streaming-scala
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-runtime
> > > > > > >>>>>>> flink-runtime-web
> > > > > > >>>>>>> flink-optimizer
> > > > > > >>>>>>> flink-clients
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-shaded
> > > > > > >>>>>>>   -> flink-shaded-hadoop
> > > > > > >>>>>>>   -> flink-shaded-hadoop2
> > > > > > >>>>>>>   -> flink-shaded-include-yarn-tests
> > > > > > >>>>>>>   -> flink-shaded-curator
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-examples
> > > > > > >>>>>>>   -> (have all examples, Scala and Java, Batch and
> > Streaming)
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-batch-connectors
> > > > > > >>>>>>>   -> flink-avro
> > > > > > >>>>>>>   -> flink-jdbc
> > > > > > >>>>>>>   -> flink-hadoop-compatibility
> > > > > > >>>>>>>   -> flink-hbase
> > > > > > >>>>>>>   -> flink-hcatalog
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-streaming-connectors
> > > > > > >>>>>>>   -> flink-connector-twitter
> > > > > > >>>>>>>   -> flink-streaming-examples
> > > > > > >>>>>>>   -> flink-connector-flume
> > > > > > >>>>>>>   -> flink-connector-kafka
> > > > > > >>>>>>>   -> flink-connector-elasticsearch
> > > > > > >>>>>>>   -> flink-connector-rabbitmq
> > > > > > >>>>>>>   -> flink-connector-filesystem
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-libraries
> > > > > > >>>>>>>   -> flink-gelly
> > > > > > >>>>>>>   -> flink-gelly-scala
> > > > > > >>>>>>>   -> flink-ml
> > > > > > >>>>>>>   -> flink-table
> > > > > > >>>>>>>   -> flink-language-binding
> > > > > > >>>>>>>   -> flink-python
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-scala-shell
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-test-utils
> > > > > > >>>>>>> flink-tests
> > > > > > >>>>>>> flink-fs-tests
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-contrib
> > > > > > >>>>>>>   -> flink-storm-compatibility
> > > > > > >>>>>>>   -> flink-storm-compatibility-examples
> > > > > > >>>>>>>   -> flink-streaming-utils
> > > > > > >>>>>>>   -> flink-tweet-inputformat
> > > > > > >>>>>>>   -> flink-operator-stats
> > > > > > >>>>>>>   -> flink-tez
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-quickstart
> > > > > > >>>>>>>   -> flink-quickstart-java
> > > > > > >>>>>>>   -> flink-quickstart-scala
> > > > > > >>>>>>>   -> flink-tez-quickstart
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-yarn
> > > > > > >>>>>>> flink-yarn-tests
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-dist
> > > > > > >>>>>>>
> > > > > > >>>>>>> flink-benchmark
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>> Let me know if that makes sense!
> > > > > > >>>>>>>
> > > > > > >>>>>>> Greetings,
> > > > > > >>>>>>> Stephan
> > > > > > >>>>>
> > > > > > >>>>
> > > > > > >>>
> > > > > > >>
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > >
> >
>

--001a11444b90a64fbb05290e9890--