Thank you for bringing new attention to this. After thinking a bit more about it, I would actually vote to NOT offer pre-build suffixed packages. So far, the demand for Scala 2.11 has been low on the mailing list, and for those who want to use it, we have a good way to enable it manually.
On Mon, Mar 16, 2015 at 12:35 PM, Alexander Alexandrov < alexander.s.alexand...@gmail.com> wrote: > Popping up this thread. The PR build passes and we need some sort of > strategy (add suffix to packages or force manual build for 2.11) before we > merge. > > 2015-03-11 19:15 GMT+01:00 Robert Metzger <rmetz...@apache.org>: > > > Thank you for opening the pull request. It looks great so far. > > > > There came up one question while discussing Alex changes in the PR: To > make > > this change really usable for our users, we need to provide our users > with > > different poms for each artifact, containing either Scala 2.10 or 2.11. > > Its basically the same thing we did for Hadoop's dependencies. > > We would then end up with the versions: 0.9.0 and 0.9.0-hadoop1 > > and artifacts like "flink-core" and "flink-core_2.11". > > So each module will generate 4 artifacts to maven central. > > > > Another option would be to keep everything as it is right now, then users > > would need to set the properties correctly when building their flink > > projects. > > > > > > > > > > On Wed, Mar 11, 2015 at 12:41 AM, Alexander Alexandrov < > > alexander.s.alexand...@gmail.com> wrote: > > > > > The PR is here: https://github.com/apache/flink/pull/477 > > > > > > Cheers! > > > > > > 2015-03-10 18:07 GMT+01:00 Alexander Alexandrov < > > > alexander.s.alexand...@gmail.com>: > > > > > > > Yes, will do. > > > > > > > > 2015-03-10 16:39 GMT+01:00 Robert Metzger <rmetz...@apache.org>: > > > > > > > >> Very nice work. > > > >> The changes are probably somewhat easy to merge. Except for the > > version > > > >> properties in the parent pom, there should not be a bigger issue. > > > >> > > > >> Can you also add additional build profiles to travis for scala 2.11 > ? > > > >> > > > >> On Tue, Mar 10, 2015 at 2:50 PM, Alexander Alexandrov < > > > >> alexander.s.alexand...@gmail.com> wrote: > > > >> > > > >> > We have is almost ready here: > > > >> > > > > >> > https://github.com/stratosphere/flink/commits/scala_2.11_rebased > > > >> > > > > >> > I wanted to open a PR today > > > >> > > > > >> > 2015-03-10 11:28 GMT+01:00 Robert Metzger <rmetz...@apache.org>: > > > >> > > > > >> > > Hey Alex, > > > >> > > > > > >> > > I don't know the exact status of the Scala 2.11 integration. > But I > > > >> wanted > > > >> > > to point you to https://github.com/apache/flink/pull/454, which > > is > > > >> > > changing > > > >> > > a huge portion of our maven build infrastructure. > > > >> > > If you haven't started yet, it might make sense to base your > > > >> integration > > > >> > > onto that pull request. > > > >> > > > > > >> > > Otherwise, let me know if you have troubles rebasing your > changes. > > > >> > > > > > >> > > On Mon, Mar 2, 2015 at 9:13 PM, Chiwan Park < > > chiwanp...@icloud.com> > > > >> > wrote: > > > >> > > > > > >> > > > +1 for Scala 2.11 > > > >> > > > > > > >> > > > Regards. > > > >> > > > Chiwan Park (Sent with iPhone) > > > >> > > > > > > >> > > > > > > >> > > > > On Mar 3, 2015, at 2:43 AM, Robert Metzger < > > rmetz...@apache.org > > > > > > > >> > > wrote: > > > >> > > > > > > > >> > > > > I'm +1 if this doesn't affect existing Scala 2.10 users. > > > >> > > > > > > > >> > > > > I would also suggest to add a scala 2.11 build to travis as > > well > > > >> to > > > >> > > > ensure > > > >> > > > > everything is working with the different Hadoop/JVM > versions. > > > >> > > > > It shouldn't be a big deal to offer scala_version x > > > hadoop_version > > > >> > > builds > > > >> > > > > for newer releases. > > > >> > > > > You only need to add more builds here: > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > https://github.com/apache/flink/blob/master/tools/create_release_files.sh#L131 > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > On Mon, Mar 2, 2015 at 6:17 PM, Till Rohrmann < > > > >> trohrm...@apache.org> > > > >> > > > wrote: > > > >> > > > > > > > >> > > > >> +1 for Scala 2.11 > > > >> > > > >> > > > >> > > > >> On Mon, Mar 2, 2015 at 5:02 PM, Alexander Alexandrov < > > > >> > > > >> alexander.s.alexand...@gmail.com> wrote: > > > >> > > > >> > > > >> > > > >>> Spark currently only provides pre-builds for 2.10 and > > requires > > > >> > custom > > > >> > > > >> build > > > >> > > > >>> for 2.11. > > > >> > > > >>> > > > >> > > > >>> Not sure whether this is the best idea, but I can see the > > > >> benefits > > > >> > > > from a > > > >> > > > >>> project management point of view... > > > >> > > > >>> > > > >> > > > >>> Would you prefer to have a {scala_version} × > > {hadoop_version} > > > >> > > > integrated > > > >> > > > >> on > > > >> > > > >>> the website? > > > >> > > > >>> > > > >> > > > >>> 2015-03-02 16:57 GMT+01:00 Aljoscha Krettek < > > > >> aljos...@apache.org>: > > > >> > > > >>> > > > >> > > > >>>> +1 I also like it. We just have to figure out how we can > > > >> publish > > > >> > two > > > >> > > > >>>> sets of release artifacts. > > > >> > > > >>>> > > > >> > > > >>>> On Mon, Mar 2, 2015 at 4:48 PM, Stephan Ewen < > > > se...@apache.org > > > >> > > > > >> > > > wrote: > > > >> > > > >>>>> Big +1 from my side! > > > >> > > > >>>>> > > > >> > > > >>>>> Does it have to be a Maven profile, or does a maven > > property > > > >> > work? > > > >> > > > >>>> (Profile > > > >> > > > >>>>> may be needed for quasiquotes dependency?) > > > >> > > > >>>>> > > > >> > > > >>>>> On Mon, Mar 2, 2015 at 4:36 PM, Alexander Alexandrov < > > > >> > > > >>>>> alexander.s.alexand...@gmail.com> wrote: > > > >> > > > >>>>> > > > >> > > > >>>>>> Hi there, > > > >> > > > >>>>>> > > > >> > > > >>>>>> since I'm relying on Scala 2.11.4 on a project I've > been > > > >> working > > > >> > > > >> on, I > > > >> > > > >>>>>> created a branch which updates the Scala version used > by > > > >> Flink > > > >> > > from > > > >> > > > >>>> 2.10.4 > > > >> > > > >>>>>> to 2.11.4: > > > >> > > > >>>>>> > > > >> > > > >>>>>> > https://github.com/stratosphere/flink/commits/scala_2.11 > > > >> > > > >>>>>> > > > >> > > > >>>>>> Everything seems to work fine and the PR contains minor > > > >> changes > > > >> > > > >>>> compared to > > > >> > > > >>>>>> Spark: > > > >> > > > >>>>>> > > > >> > > > >>>>>> https://issues.apache.org/jira/browse/SPARK-4466 > > > >> > > > >>>>>> > > > >> > > > >>>>>> If you're interested, I can rewrite this as a Maven > > Profile > > > >> and > > > >> > > > >> open a > > > >> > > > >>>> PR > > > >> > > > >>>>>> so people can build Flink with 2.11 support. > > > >> > > > >>>>>> > > > >> > > > >>>>>> I suggest to do this sooner rather than later in order > to > > > >> > > > >>>>>> > > > >> > > > >>>>>> * the number of code changes enforced by migration > small > > > and > > > >> > > > >>> tractable; > > > >> > > > >>>>>> * discourage the use of deprecated or 2.11-incompatible > > > >> source > > > >> > > code > > > >> > > > >> in > > > >> > > > >>>>>> future commits; > > > >> > > > >>>>>> > > > >> > > > >>>>>> Regards, > > > >> > > > >>>>>> A. > > > >> > > > >>>>>> > > > >> > > > >>>> > > > >> > > > >>> > > > >> > > > >> > > > >> > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > > > > > > > > >
