Re: [PSA] Apache Spark uses GitHub Actions to run the tests

2020-07-23 Thread Hyukjin Kwon
Ah, it doesn’t have an integration with the PR dashboard for now. Imran, have you set up this ? Once you have the write access directly to the GitHub repo, I think you will be able to see the “Re-run jobs” button: [image: Screen Shot 2020-07-24 at

Re: [PSA] Apache Spark uses GitHub Actions to run the tests

2020-07-23 Thread Imran Rashid
Thanks for setting this up, Hyukjin. How do you re-trigger tests in github actions? Eg. there is a failure that appears to be some random infra thing or a flaky test, or maybe the tests were just run a while back so you want to get a fresh batch of tests. I think the old "Jenkins, retest this

Re: [DISCUSS] Amend the commiter guidelines on the subject of -1s & how we expect PR discussion to be treated.

2020-07-23 Thread Mridul Muralidharan
Thanks Holden, this version looks good to me. +1 Regards, Mridul On Thu, Jul 23, 2020 at 3:56 PM Imran Rashid wrote: > Sure, that sounds good to me. +1 > > On Wed, Jul 22, 2020 at 1:50 PM Holden Karau wrote: > >> >> >> On Wed, Jul 22, 2020 at 7:39 AM Imran Rashid < iras...@apache.org > >>

Re: [DISCUSS] Amend the commiter guidelines on the subject of -1s & how we expect PR discussion to be treated.

2020-07-23 Thread Imran Rashid
Sure, that sounds good to me. +1 On Wed, Jul 22, 2020 at 1:50 PM Holden Karau wrote: > > > On Wed, Jul 22, 2020 at 7:39 AM Imran Rashid < iras...@apache.org > wrote: > >> Hi Holden, >> >> thanks for leading this discussion, I'm in favor in general. I have one >> specific question -- these two

Re: Exposing Spark parallelized directory listing & non-locality listing in core

2020-07-23 Thread Holden Karau
Awesome that sounds great :) On Thu, Jul 23, 2020 at 3:43 AM Steve Loughran wrote: > > > On Wed, 22 Jul 2020 at 18:50, Holden Karau wrote: > >> Wonderful. To be clear the patch is more to start the discussion about >> how we want to do it and less what I think is the right way. >> >> > be

Re: Exposing Spark parallelized directory listing & non-locality listing in core

2020-07-23 Thread Steve Loughran
On Wed, 22 Jul 2020 at 18:50, Holden Karau wrote: > Wonderful. To be clear the patch is more to start the discussion about how > we want to do it and less what I think is the right way. > > be happy to give a quick online tour of ongoing work on S3A enhancements some time next week, get feedback