You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spark.apache.org by Hyukjin Kwon <gu...@gmail.com> on 2016/05/06 15:45:36 UTC

Proposal of closing some PRs and maybe some PRs abandoned by its author

Hi all,


This was similar with the proposal of closing PRs before I asked.

I think the PRs suggested to be closed below are closable but not very sure
of PRs apparently abandoned by its author at least for a month.

I remember the discussion about auto-closing PR before. So, I included the
PRs as below anyway.

I looked though the open every PR at this time and could make a list as
below:


1. Suggested to be closed.


https://github.com/apache/spark/pull/7739  <- not sure

https://github.com/apache/spark/pull/9354

https://github.com/apache/spark/pull/9451

https://github.com/apache/spark/pull/10507

https://github.com/apache/spark/pull/10486

https://github.com/apache/spark/pull/10460

https://github.com/apache/spark/pull/10967

https://github.com/apache/spark/pull/10945

https://github.com/apache/spark/pull/10701

https://github.com/apache/spark/pull/10681

https://github.com/apache/spark/pull/11766



2. Author not answering at least for a month.


https://github.com/apache/spark/pull/9907

https://github.com/apache/spark/pull/9920

https://github.com/apache/spark/pull/9936

https://github.com/apache/spark/pull/10052

https://github.com/apache/spark/pull/10125

https://github.com/apache/spark/pull/10209

https://github.com/apache/spark/pull/10572 <- not sure

https://github.com/apache/spark/pull/10326

https://github.com/apache/spark/pull/10379

https://github.com/apache/spark/pull/10403

https://github.com/apache/spark/pull/10466

https://github.com/apache/spark/pull/10572 <- not sure

https://github.com/apache/spark/pull/10995

https://github.com/apache/spark/pull/10887

https://github.com/apache/spark/pull/10842

https://github.com/apache/spark/pull/11005

https://github.com/apache/spark/pull/11036

https://github.com/apache/spark/pull/11129

https://github.com/apache/spark/pull/11610

https://github.com/apache/spark/pull/11729

https://github.com/apache/spark/pull/11980

https://github.com/apache/spark/pull/12075


Thanks.

Re: Proposal of closing some PRs and maybe some PRs abandoned by its author

Posted by Nicholas Chammas <ni...@gmail.com>.
Alex has built tooling for this btw:
https://github.com/databricks/spark-pr-dashboard/pull/71

On Fri, May 6, 2016 at 12:15 PM Ted Yu <yu...@gmail.com> wrote:

> PR #10572 was listed twice.
>
> In the future, is it possible to include the contributor's handle beside
> the PR number so that people can easily recognize their own PR ?
>
> Thanks
>
> On Fri, May 6, 2016 at 8:45 AM, Hyukjin Kwon <gu...@gmail.com> wrote:
>
>> Hi all,
>>
>>
>> This was similar with the proposal of closing PRs before I asked.
>>
>> I think the PRs suggested to be closed below are closable but not very
>> sure of PRs apparently abandoned by its author at least for a month.
>>
>> I remember the discussion about auto-closing PR before. So, I included
>> the PRs as below anyway.
>>
>> I looked though the open every PR at this time and could make a list as
>> below:
>>
>>
>> 1. Suggested to be closed.
>>
>>
>> https://github.com/apache/spark/pull/7739  <- not sure
>>
>> https://github.com/apache/spark/pull/9354
>>
>> https://github.com/apache/spark/pull/9451
>>
>> https://github.com/apache/spark/pull/10507
>>
>> https://github.com/apache/spark/pull/10486
>>
>> https://github.com/apache/spark/pull/10460
>>
>> https://github.com/apache/spark/pull/10967
>>
>> https://github.com/apache/spark/pull/10945
>>
>> https://github.com/apache/spark/pull/10701
>>
>> https://github.com/apache/spark/pull/10681
>>
>> https://github.com/apache/spark/pull/11766
>>
>>
>>
>> 2. Author not answering at least for a month.
>>
>>
>> https://github.com/apache/spark/pull/9907
>>
>> https://github.com/apache/spark/pull/9920
>>
>> https://github.com/apache/spark/pull/9936
>>
>> https://github.com/apache/spark/pull/10052
>>
>> https://github.com/apache/spark/pull/10125
>>
>> https://github.com/apache/spark/pull/10209
>>
>> https://github.com/apache/spark/pull/10572 <- not sure
>>
>> https://github.com/apache/spark/pull/10326
>>
>> https://github.com/apache/spark/pull/10379
>>
>> https://github.com/apache/spark/pull/10403
>>
>> https://github.com/apache/spark/pull/10466
>>
>> https://github.com/apache/spark/pull/10572 <- not sure
>>
>> https://github.com/apache/spark/pull/10995
>>
>> https://github.com/apache/spark/pull/10887
>>
>> https://github.com/apache/spark/pull/10842
>>
>> https://github.com/apache/spark/pull/11005
>>
>> https://github.com/apache/spark/pull/11036
>>
>> https://github.com/apache/spark/pull/11129
>>
>> https://github.com/apache/spark/pull/11610
>>
>> https://github.com/apache/spark/pull/11729
>>
>> https://github.com/apache/spark/pull/11980
>>
>> https://github.com/apache/spark/pull/12075
>>
>>
>> Thanks.
>>
>>
>>
>

Re: Proposal of closing some PRs and maybe some PRs abandoned by its author

Posted by Ted Yu <yu...@gmail.com>.
PR #10572 was listed twice.

In the future, is it possible to include the contributor's handle beside
the PR number so that people can easily recognize their own PR ?

Thanks

On Fri, May 6, 2016 at 8:45 AM, Hyukjin Kwon <gu...@gmail.com> wrote:

> Hi all,
>
>
> This was similar with the proposal of closing PRs before I asked.
>
> I think the PRs suggested to be closed below are closable but not very
> sure of PRs apparently abandoned by its author at least for a month.
>
> I remember the discussion about auto-closing PR before. So, I included the
> PRs as below anyway.
>
> I looked though the open every PR at this time and could make a list as
> below:
>
>
> 1. Suggested to be closed.
>
>
> https://github.com/apache/spark/pull/7739  <- not sure
>
> https://github.com/apache/spark/pull/9354
>
> https://github.com/apache/spark/pull/9451
>
> https://github.com/apache/spark/pull/10507
>
> https://github.com/apache/spark/pull/10486
>
> https://github.com/apache/spark/pull/10460
>
> https://github.com/apache/spark/pull/10967
>
> https://github.com/apache/spark/pull/10945
>
> https://github.com/apache/spark/pull/10701
>
> https://github.com/apache/spark/pull/10681
>
> https://github.com/apache/spark/pull/11766
>
>
>
> 2. Author not answering at least for a month.
>
>
> https://github.com/apache/spark/pull/9907
>
> https://github.com/apache/spark/pull/9920
>
> https://github.com/apache/spark/pull/9936
>
> https://github.com/apache/spark/pull/10052
>
> https://github.com/apache/spark/pull/10125
>
> https://github.com/apache/spark/pull/10209
>
> https://github.com/apache/spark/pull/10572 <- not sure
>
> https://github.com/apache/spark/pull/10326
>
> https://github.com/apache/spark/pull/10379
>
> https://github.com/apache/spark/pull/10403
>
> https://github.com/apache/spark/pull/10466
>
> https://github.com/apache/spark/pull/10572 <- not sure
>
> https://github.com/apache/spark/pull/10995
>
> https://github.com/apache/spark/pull/10887
>
> https://github.com/apache/spark/pull/10842
>
> https://github.com/apache/spark/pull/11005
>
> https://github.com/apache/spark/pull/11036
>
> https://github.com/apache/spark/pull/11129
>
> https://github.com/apache/spark/pull/11610
>
> https://github.com/apache/spark/pull/11729
>
> https://github.com/apache/spark/pull/11980
>
> https://github.com/apache/spark/pull/12075
>
>
> Thanks.
>
>
>

Re: Proposal of closing some PRs and maybe some PRs abandoned by its author

Posted by Hyukjin Kwon <gu...@gmail.com>.
Sorry, here are two more PRs I guess should be closed.

10052  dereksabryfb   - This one is suggested by Sean and Herman to close
10356  somideshmukh   - This has not been confirmed by a committer yet but
I tested this and it seems not an issue.

Thanks.

2016-05-07 2:47 GMT+09:00 Sean Owen <so...@cloudera.com>:

> Skimmed these as a one-off exercise, and I suggest ...
>
> Leave open for now
>
> 7739 mgrover
> 10701 nraychaudhuri
> 9920 jliwork
> 9936 Lewuathe
> 10052 dereksabryfb
> 10125 kevinyu98
> 10466 hhbyyh
> 10995 tedyu
> 10887 blbradley
> 11005 huaxingao
> 11129 AtkinsChang
> 11610 iyounus
> 11729 mbaddar1
> 11980 koertkuipers
> 12075 zhuoliu
> 10572 navis
> 10945 dmarcous
>
> Close
>
> 9354 jacek-lewandowski
> 9451 vidma
> 10507 JerryLead
> 10486 wilson888888888
> 10460 huaxingao
> 10967 kevinyu98
> 10681 nikit-os
> 11766 s4weng
> 9907 Lewuathe
> 10209 nongli
> 10379 yanakad
> 10403 naveenminchu
> 10842 rajeshbalamohan
> 11036 mbautin
>
> On Fri, May 6, 2016 at 4:45 PM, Hyukjin Kwon <gu...@gmail.com> wrote:
> > Hi all,
> >
> >
> > This was similar with the proposal of closing PRs before I asked.
> >
> > I think the PRs suggested to be closed below are closable but not very
> sure
> > of PRs apparently abandoned by its author at least for a month.
> >
> > I remember the discussion about auto-closing PR before. So, I included
> the
> > PRs as below anyway.
> >
> > I looked though the open every PR at this time and could make a list as
> > below:
> >
> >
> > 1. Suggested to be closed.
> >
> >
> > https://github.com/apache/spark/pull/7739  <- not sure
> >
> > https://github.com/apache/spark/pull/9354
> >
> > https://github.com/apache/spark/pull/9451
> >
> > https://github.com/apache/spark/pull/10507
> >
> > https://github.com/apache/spark/pull/10486
> >
> > https://github.com/apache/spark/pull/10460
> >
> > https://github.com/apache/spark/pull/10967
> >
> > https://github.com/apache/spark/pull/10945
> >
> > https://github.com/apache/spark/pull/10701
> >
> > https://github.com/apache/spark/pull/10681
> >
> > https://github.com/apache/spark/pull/11766
> >
> >
> >
> > 2. Author not answering at least for a month.
> >
> >
> > https://github.com/apache/spark/pull/9907
> >
> > https://github.com/apache/spark/pull/9920
> >
> > https://github.com/apache/spark/pull/9936
> >
> > https://github.com/apache/spark/pull/10052
> >
> > https://github.com/apache/spark/pull/10125
> >
> > https://github.com/apache/spark/pull/10209
> >
> > https://github.com/apache/spark/pull/10572 <- not sure
> >
> > https://github.com/apache/spark/pull/10326
> >
> > https://github.com/apache/spark/pull/10379
> >
> > https://github.com/apache/spark/pull/10403
> >
> > https://github.com/apache/spark/pull/10466
> >
> > https://github.com/apache/spark/pull/10572 <- not sure
> >
> > https://github.com/apache/spark/pull/10995
> >
> > https://github.com/apache/spark/pull/10887
> >
> > https://github.com/apache/spark/pull/10842
> >
> > https://github.com/apache/spark/pull/11005
> >
> > https://github.com/apache/spark/pull/11036
> >
> > https://github.com/apache/spark/pull/11129
> >
> > https://github.com/apache/spark/pull/11610
> >
> > https://github.com/apache/spark/pull/11729
> >
> > https://github.com/apache/spark/pull/11980
> >
> > https://github.com/apache/spark/pull/12075
> >
> >
> > Thanks.
> >
> >
>

Re: Proposal of closing some PRs and maybe some PRs abandoned by its author

Posted by Sean Owen <so...@cloudera.com>.
Skimmed these as a one-off exercise, and I suggest ...

Leave open for now

7739 mgrover
10701 nraychaudhuri
9920 jliwork
9936 Lewuathe
10052 dereksabryfb
10125 kevinyu98
10466 hhbyyh
10995 tedyu
10887 blbradley
11005 huaxingao
11129 AtkinsChang
11610 iyounus
11729 mbaddar1
11980 koertkuipers
12075 zhuoliu
10572 navis
10945 dmarcous

Close

9354 jacek-lewandowski
9451 vidma
10507 JerryLead
10486 wilson888888888
10460 huaxingao
10967 kevinyu98
10681 nikit-os
11766 s4weng
9907 Lewuathe
10209 nongli
10379 yanakad
10403 naveenminchu
10842 rajeshbalamohan
11036 mbautin

On Fri, May 6, 2016 at 4:45 PM, Hyukjin Kwon <gu...@gmail.com> wrote:
> Hi all,
>
>
> This was similar with the proposal of closing PRs before I asked.
>
> I think the PRs suggested to be closed below are closable but not very sure
> of PRs apparently abandoned by its author at least for a month.
>
> I remember the discussion about auto-closing PR before. So, I included the
> PRs as below anyway.
>
> I looked though the open every PR at this time and could make a list as
> below:
>
>
> 1. Suggested to be closed.
>
>
> https://github.com/apache/spark/pull/7739  <- not sure
>
> https://github.com/apache/spark/pull/9354
>
> https://github.com/apache/spark/pull/9451
>
> https://github.com/apache/spark/pull/10507
>
> https://github.com/apache/spark/pull/10486
>
> https://github.com/apache/spark/pull/10460
>
> https://github.com/apache/spark/pull/10967
>
> https://github.com/apache/spark/pull/10945
>
> https://github.com/apache/spark/pull/10701
>
> https://github.com/apache/spark/pull/10681
>
> https://github.com/apache/spark/pull/11766
>
>
>
> 2. Author not answering at least for a month.
>
>
> https://github.com/apache/spark/pull/9907
>
> https://github.com/apache/spark/pull/9920
>
> https://github.com/apache/spark/pull/9936
>
> https://github.com/apache/spark/pull/10052
>
> https://github.com/apache/spark/pull/10125
>
> https://github.com/apache/spark/pull/10209
>
> https://github.com/apache/spark/pull/10572 <- not sure
>
> https://github.com/apache/spark/pull/10326
>
> https://github.com/apache/spark/pull/10379
>
> https://github.com/apache/spark/pull/10403
>
> https://github.com/apache/spark/pull/10466
>
> https://github.com/apache/spark/pull/10572 <- not sure
>
> https://github.com/apache/spark/pull/10995
>
> https://github.com/apache/spark/pull/10887
>
> https://github.com/apache/spark/pull/10842
>
> https://github.com/apache/spark/pull/11005
>
> https://github.com/apache/spark/pull/11036
>
> https://github.com/apache/spark/pull/11129
>
> https://github.com/apache/spark/pull/11610
>
> https://github.com/apache/spark/pull/11729
>
> https://github.com/apache/spark/pull/11980
>
> https://github.com/apache/spark/pull/12075
>
>
> Thanks.
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@spark.apache.org
For additional commands, e-mail: dev-help@spark.apache.org