Suspicious entries: * C++ API Ivan Veselovsky * Docker, Mesos, YARN integration Igor Sapego
On Mon, Mar 21, 2016 at 6:08 PM, Sergi Vladykin <sergi.vlady...@gmail.com> wrote: > Looks good. > > Sergi > > 2016-03-21 16:37 GMT+03:00 Denis Magda <dma...@gridgain.com>: > > > Igniters, > > > > I've prepared a draft of the maintainers list. > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#HowtoContribute-ReviewProcessandMaintainers > > > > Please review it and/or adjust it whenever is needed. > > > > If you have any thoughts, concerns let's discuss them there. > > > > -- > > Denis > > > > On 3/10/2016 1:37 PM, Sergi Vladykin wrote: > > > >> If everyone is ok with the proposals, then we need to set this new > >> approach > >> and properly document it. > >> > >> Also we need to select list of RTC modules and elect their maintainers. > >> > >> Sergi > >> > >> 2016-03-05 19:31 GMT+03:00 Sergi Vladykin <sergi.vlady...@gmail.com>: > >> > >> +1 to the original proposal of Denis to introduce module maintainers and > >>> RTC process > >>> +1 to the proposal of Raul to restrict number of core modules, which > >>> require maintainers review > >>> > >>> Sergi > >>> > >>> > >>> 2016-03-05 6:43 GMT+03:00 Konstantin Boudnik <c...@apache.org>: > >>> > >>> It saddens me to see this coming to it ;( > >>>> > >>>> On Thu, Mar 03, 2016 at 02:54PM, Denis Magda wrote: > >>>> > >>>>> Igniters, > >>>>> > >>>>> I would propose to switch back to review-then-commit process. This > >>>>> process has to be followed by both contributors and committers. > >>>>> > >>>>> There is a reason for this I have in mind. Ignite is a complex > >>>>> platform with several big modules. Some of the people may be experts > >>>>> in module A while others in module B etc. > >>>>> If a committer, who is good in module A, makes changes in module B > >>>>> merging the changes without a review this can break module's B > >>>>> internal functionality that the committer didn't take into account. > >>>>> > >>>>> My proposal is to introduce a list of maintainers for every Ignite > >>>>> module like it's done in Spark [1] and a rule that will require a > >>>>> committer to get an approval from a module maintainer before merging > >>>>> changes. > >>>>> > >>>>> Thoughts? > >>>>> > >>>>> -- > >>>>> Denis > >>>>> > >>>>> [1] > >>>>> > >>>> > >>>> > https://cwiki.apache.org/confluence/display/SPARK/Committers#Committers-ReviewProcessandMaintainers > >>>> > >>>>> > >>>>> > >>>>> > >>> > > >