Dmitriy, admin rights allow closing any PRs, I wonder who is admin for Ignite GitHub mirror.
On Tue, Dec 11, 2018 at 1:29 PM Павлухин Иван <vololo...@gmail.com> wrote: > Dmitiy, > > I see the point. And also I see that not everyone supports a suggested > cleanup. (I personally like the idea very much). But all in all I > think that everyone is free to close his own PRs if he would like to. > As already have done I, Pavel and others. > > Nikolay, > > > 3. I dont' see any issues with many opened PRs. > > What problem we trying to solve? > > From my point of view the main point is visibility. And in my previous > experience going through open PRs was very good matter of > understanding what is REALLY happening in the project. With 1K+ open > PRs I doubt that it is feasible. > > Also, in Cassandra they have about 100 open PRs. So, it looks like > that different communities have different approaches. And we can > develop our own. > вт, 11 дек. 2018 г. в 13:02, Pavel Kovalenko <jokse...@gmail.com>: > > > > +1 > > > > When I just joined the project I was really confused with the huge number > > of opened PRs with poor description and cryptic reason. > > This can give to newcomers a weird representation of what is > > actually happening in the project. > > Don't be lazy, just spend 5-10 minutes of free time and clean up after > you. > > > > вт, 11 дек. 2018 г. в 10:16, Dmitriy Pavlov <dpav...@apache.org>: > > > > > My point is - close your outdated PRs/if you don't need. It is not a > > > pattern nor enforcement nor bureaucracy. > > > > > > You're absolutely free to use PRs in any way you like to (e.g. > experiments > > > with TC runs). > > > > > > Now count of PRs shocks newcomers. > > > > > > вт, 11 дек. 2018 г. в 10:10, Nikolay Izhikov <nizhi...@apache.org>: > > > > > > > Hello, Ivan. > > > > > > > > Personally, I keep my PR's clear. > > > > So, I don't have dozens of opened PR. > > > > > > > > But, I don't support Dmitriy proposal for several reasons: > > > > > > > > 1. We introduce some new, not required, level of bureaucracy. > > > > From my experience - not required bureaucracy is a BAD thing. > > > > > > > > 2. We spread our work pattern to whole community. > > > > I believe there are many patterns of dealing with *YOUR OWN* PRs. > > > > Some of them can lead to dozens of opened PRs to master. > > > > Whats wrong with it? > > > > > > > > 3. I dont' see any issues with many opened PRs. > > > > What problem we trying to solve? > > > > > > > > 4. Closing abanodned PRs doesn't force anybody to review the rest. > > > > Instead of ordering something to one way or another, let's solve real > > > > problem: > > > > > > > > - help the community doing PR review. > > > > - fixing failing tests. > > > > - introducing new code inspections to make our code base > clear. > > > > - making Ignite improvements > > > > > > > > 5. I don't see how our numbers differs from other Apache projects > > > > > > > > Apache Kafka - 533 PR opened. > > > > Apache Spark - 484 PR opened. > > > > Apache Flink - 430 PR opened. > > > > > > > > В Вт, 11/12/2018 в 09:24 +0300, Pavel Tupitsyn пишет: > > > > > Agree with Dmitriy. > > > > > > > > > > We use GitHub PRs in our workflow, therefore we should keep them in > > > > order. > > > > > > > > > > We can close PRs that refer to closed tickets, this can be done > with a > > > > > simple script. > > > > > > > > > > On Tue, Dec 11, 2018 at 9:15 AM Павлухин Иван <vololo...@gmail.com > > > > > > wrote: > > > > > > > > > > > Nikolay, > > > > > > > > > > > > I must say that when I first saw 1K+ open PRs my first thought > was > > > > > > that something was wrong with a review process. In my mind in not > > > very > > > > > > big project open PR list can reflect very well the real work in > > > > > > progress. For bigger projects things become more complicated. > > > > > > > > > > > > Dmitriy, > > > > > > > > > > > > Do you have some cleanup automation in mind? Immediately I think > that > > > > > > it is fully safe to close all PRs that were not touched more > than a > > > > > > year. > > > > > > пн, 10 дек. 2018 г. в 20:01, Dmitriy Pavlov <dpav...@apache.org > >: > > > > > > > > > > > > > > The main concern is related to chances that newcomer will have > to > > > > obtain > > > > > > > > > > > > a > > > > > > > review support from the community. > > > > > > > > > > > > > > Actually, a lot of people doing their best to provide a > feedback to > > > > > > > newcomers, and count of issues still in PA state goes down (84 > is a > > > > > > > relatively small count of issues in PA state). But 1428 PRs may > > > > imply we > > > > > > > don't review here, as we have tons of incomplete PRs. Actually, > > > most > > > > of > > > > > > > these PRs were merged (but not using ./apply-pull-request.sh > > > script, > > > > but > > > > > > > manually, without reference to PRs). > > > > > > > > > > > > > > Another benefit of revising this list, if there are any changes > > > which > > > > > > > were not accomplished with a proper ticket with PA status, we > will > > > > > > > > > > > > identify > > > > > > > a number of additional contributions to be applied to the > codebase. > > > > > > > > > > > > > > > > > > > > > пн, 10 дек. 2018 г. в 19:53, Nikolay Izhikov < > nizhi...@apache.org > > > >: > > > > > > > > > > > > > > > Hello, Dmitriy. > > > > > > > > > > > > > > > > What, exactly concerns newcomers? > > > > > > > > What is wrong with opened PR? > > > > > > > > How project will benefit from closed PR? > > > > > > > > > > > > > > > > > The same proposal is related to IEP statuses. If you were > > > > involved > > > > > > > > > > > > in an > > > > > > > > IEP, please validate its status > > > > > > > > > > > > > > > > +1. We should maintain IEP description up to date. > > > > > > > > > > > > > > > > пн, 10 дек. 2018 г., 19:15 Dmitriy Pavlov dpav...@apache.org > : > > > > > > > > > > > > > > > > > Hi Igniters, > > > > > > > > > > > > > > > > > > Newcomers to Apache Ignite sometimes became concerned about > > > many > > > > > > > > > > > > open PRs > > > > > > > > > in the project. Apache Ignite TC Bot also performs runs > checks > > > > with > > > > > > > > > > > > a PR > > > > > > > > > open. Apache Ignite pulls list > > > > > > > > > > > > https://github.com/apache/ignite/pulls > > > > > > > > > contains > > > > > > > > > 1442 PRs open while only 84 issues are waiting for review. > > > > > > > > > > > > > > > > > > Could you please verify the list of your PRs in Apache > Ignite > > > > > > > > > https://github.com/pulls and close every not > needed/already > > > > merged > > > > > > > > > change? > > > > > > > > > > > > > > > > > > The same proposal is related to IEP statuses. If you were > > > > involved > > > > > > > > > > > > in an > > > > > > > > > IEP, please validate its status here > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Active+Proposals > > > > > > > > > Please > > > > > > > > > set a correct state for your IEP, as well. > > > > > > > > > > > > > > > > > > Should you have any questions, please don't hesitate to ask > > > here. > > > > > > > > > > > > Thank > > > > > > > > you > > > > > > > > > in advance! > > > > > > > > > > > > > > > > > > Sincerely, > > > > > > > > > Dmitriy Pavlov > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Best regards, > > > > > > Ivan Pavlukhin > > > > > > > > > > > > > > > > > -- > Best regards, > Ivan Pavlukhin >
