Anton, We already had a discussion about this: http://apache-ignite-developers.2346864.n4.nabble.com/ignite-spark-module-in-Hadoop-Accelerator-td12343.html
While I'm personally very much for getting rid of two separate editions, we didn't come to consensus there. -Val On Wed, Feb 7, 2018 at 2:13 AM, Anton Vinogradov <a...@apache.org> wrote: > Denis, Petr, > > I checked PR and found we have *overcomplicated* logic with "fabric" and > "hadoop" postfixs. > > Do we really need to assembly 2 editions? > "Hadoop" edition still valued? > > My proposal is to get rid of "hadoop" edition and replace it with > instruction of how to use "fabric" edition instead. > Instruction will be pretty easy -> move "hadoop" folder from "optional" to > root directory :) > > In that case we can just remove all postfix logic from maven poms and > simplify release process. > > On Thu, Dec 28, 2017 at 9:20 PM, Denis Magda <dma...@apache.org> wrote: > > > Petr, thanks for solving it! > > > > Hope that Anton V. or some other build master will double-check the > > changes and merge them. > > > > — > > Denis > > > > > On Dec 28, 2017, at 8:29 AM, Petr Ivanov <mr.wei...@gmail.com> wrote: > > > > > > IGNITE-7251 is done, needs review and some additional tests. See PR > > #3315 [1]. > > > > > > > > > [1] https://github.com/apache/ignite/pull/3315 < > > https://github.com/apache/ignite/pull/3315> > > > > > > > > > > > >> On 20 Dec 2017, at 23:15, Denis Magda <dma...@apache.org> wrote: > > >> > > >> Petr, thanks, such a swift turnaround! > > >> > > >> Have you found the one who can asses and review the changes? > > >> > > >> Maintainers label might be helpful. Just ping them directly: > > >> https://cwiki.apache.org/confluence/display/IGNITE/How+ > > to+Contribute#HowtoContribute-ReviewProcessandMaintainers < > > https://cwiki.apache.org/confluence/display/IGNITE/How+ > > to+Contribute#HowtoContribute-ReviewProcessandMaintainers> > > >> > > >> > > >> — > > >> Denis > > >> > > >>> On Dec 20, 2017, at 12:24 AM, Petr Ivanov <mr.wei...@gmail.com> > wrote: > > >>> > > >>> Assigned myself — done the same work while preparing RPM package. > > >>> But for fixing DEVNOTES.txt waiting for review and merge of > > IGNITE-7107 [1]. > > >>> > > >>> > > >>> [1] https://issues.apache.org/jira/browse/IGNITE-7107 > > >>> > > >>> > > >>> > > >>>> On 19 Dec 2017, at 22:55, Denis Magda <dma...@apache.org> wrote: > > >>>> > > >>>> All the bids were accepted and the verdict is executed: > > >>>> https://issues.apache.org/jira/browse/IGNITE-7251 < > > https://issues.apache.org/jira/browse/IGNITE-7251> > > >>>> > > >>>> Who is ready to pick this up? > > >>>> > > >>>> — > > >>>> Denis > > >>>> > > >>>>> On Dec 19, 2017, at 5:35 AM, Anton Vinogradov < > > avinogra...@gridgain.com> wrote: > > >>>>> > > >>>>> +1б фо шур > > >>>>> > > >>>>> On Tue, Dec 19, 2017 at 9:59 AM, Vladimir Ozerov < > > voze...@gridgain.com> > > >>>>> wrote: > > >>>>> > > >>>>>> +1б вуаштшеудн > > >>>>>> > > >>>>>> On Tue, Dec 19, 2017 at 2:34 AM, Valentin Kulichenko < > > >>>>>> valentin.kuliche...@gmail.com> wrote: > > >>>>>> > > >>>>>>> +1 > > >>>>>>> > > >>>>>>> On Mon, Dec 18, 2017 at 12:24 PM, Dmitriy Setrakyan < > > >>>>>> dsetrak...@apache.org > > >>>>>>>> > > >>>>>>> wrote: > > >>>>>>> > > >>>>>>>> +1 (completely agree) > > >>>>>>>> > > >>>>>>>> On Mon, Dec 18, 2017 at 10:21 AM, Denis Magda < > dma...@apache.org> > > >>>>>> wrote: > > >>>>>>>> > > >>>>>>>>> Igniters, > > >>>>>>>>> > > >>>>>>>>> Apache Ignite binary releases still include “fabric” word in > > their > > >>>>>>> names: > > >>>>>>>>> https://ignite.apache.org/download.cgi#binaries < > > >>>>>>>>> https://ignite.apache.org/download.cgi#binaries> > > >>>>>>>>> > > >>>>>>>>> For instance, this is a full name of the previous release - > > >>>>>>>>> apache-ignite-fabric-2.3.0-bin. > > >>>>>>>>> It’s a little oversight on our side because the project has not > > been > > >>>>>>>>> positioned as a fabric for a while. > > >>>>>>>>> > > >>>>>>>>> Proposal! Remove “fabric” from the name and have the binary > > releases > > >>>>>>>> named > > >>>>>>>>> as - apache-ignite-{version}-bin. > > >>>>>>>>> > > >>>>>>>>> If we’re in consensus then let’s make the change in 2.4. > > >>>>>>>>> > > >>>>>>>>> — > > >>>>>>>>> Denis > > >>>>>>>>> > > >>>>>>>> > > >>>>>>> > > >>>>>> > > >>>> > > >>> > > >> > > > > > > > >
