On 6/2/20 9:15 PM, David Mollitor wrote:
I use a personal account for GitHub and it's not synced with my official
Apache account. How do I go about registering my Apache account with
GitHub so I can merge through their interface?
IIRC I've linked my account by using this interface:
https://gitbox.apache.org/setup/
In the meanwhile, can you assist with a merge here? :)
sure; I think you should also add dmolli...@apache.org as a secondary email to
your github account
About the open pr stuff: I still think our best approach of handling those things would be to close most of that 400 or so PRs...easiest would be to install the bot (at
least temporarily)
https://issues.apache.org/jira/browse/HIVE-23590
what do you think?
cheers,
Zoltan
https://github.com/apache/hive/pull/1045
Thanks!
On Tue, Jun 2, 2020 at 10:21 AM Zoltan Haindrich <k...@rxd.hu> wrote:
On 6/2/20 3:10 PM, David Mollitor wrote:
I think we might want to take one manual pass across the board. It will
most likely take more than 7 days to get through them all, so it may be
closing things that are legitimate.
yeah...a manual pass would be good; I went thru around 10 or so before
I've wrote the first mail in this thread...
and I definetly don't want to go thru 400 - so I would preffer the bot :D
One low hanging fruit (that applied to one of my PRs). The JIRA it was
associated with was already closed. Is there a way to target those?
yes; there might be certainly a lot of those...(that's why I've estimate
to 1/3 to be applicable)
but filtering out even this is an awful lot of work (or it might involve
writing a "bot")...
if it's important enough the contributor could reopen / rebase the patch.
We could try to communicate the non-hostaile intention in the message
placed by the bot.
The current message is the stale PRs would get is:
"This pull request has been automatically marked as stale because it has
not had recent activity. It will be closed if no further activity occurs."
Also, I have submitted my first PR to test out the new system. It
has passed tests. Ashutoshc has generously provided a +1. What's the
next step to get it merged into the master? Do I download the patch from
Github and apply manually using my Apache credentials? Is the "merge"
feature setup in Github? As I understand it, GitHub is only mirroring
the
Apache git system. Whatever the process we need an update in the
HowToContribute docs.
That's an interesting question; the github repo is linked to the apache
repo - so you may push/merge/whatever on the github interface; it will work.
Github supports 3 modes to merge PRs:
* We should definetly disable the "merge" option as that will just create
a internation railways station from our history :)
* rebase doesn't make it easier for reviewier to keep track new
changes...because the PR owner have to continuosly force push the branch
* squash merge work great - and I remembered that it changes the author to
the user pushing the "squash" button; however right now it seems that it
changes the author to
the "user who opened the pr" which looks good-enough for me!
(I've added the neccessary .asf.yaml changes to the existing PR)
cheers,
Zoltan
https://github.com/apache/hive/pull/1045
https://cwiki.apache.org/confluence/display/Hive/HowToContribute#HowToContribute-ApplyingaPatch
Thanks!
On Tue, Jun 2, 2020 at 4:58 AM Zoltan Haindrich <k...@rxd.hu> wrote:
I think to use "probot" we would need to ask infra to configure the
"probot" github app.
It seems to me that the stale plugin from github actions provides almost
the same functionaluty - as they seem to be more or less identical in
features I would go with the
latter.
I've opened a pr to enable stale on the hive repo:
by default it will mark as stale afte 60 days; and close it if there is
no
activity for another 7 days
https://github.com/apache/hive/pull/1049
cheers,
Zoltan
On 6/2/20 5:00 AM, Ashutosh Chauhan wrote:
How about using stalebot : https://github.com/probot/stale ?
On Mon, Jun 1, 2020 at 12:20 PM Zoltán Haindrich <k...@rxd.hu> wrote:
Hey David,
On June 1, 2020 3:52:05 PM GMT+02:00, David Mollitor <
dam6...@gmail.com
wrote:
Any idea how long it will take to run precomit on all existing PRs?
I'm not entirely sure, but a rough estimate could be:
* not every pr is mergeable; there are many which was already
merged/outdated/etc...lets estimate that 1/3 is mergeable
* every pr runs for at least 1 hours
this would mean 430/3*1/24 days of test execution which is at least 6
days.
I see little to no value in running tests on archaic prs.
We could also configure an automatism to close prs after some time of
inactivity
https://github.com/actions/stale/blob/master/README.md
The good side of this is that it will get rid of ancient prs; however
it
might seem rude to a contributor in case he is waiting for feedback or
something....
Even with that argument I think we should configure it at least for a
few
days to get rid of the dangling prs of almost a decade ! (pr#2 is
opened in
2011)...
What do you think?
cheers,
Zoltan
On Mon, Jun 1, 2020 at 9:49 AM Panos Garefalakis <panga...@gmail.com
wrote:
Same here, however, there are still ~ 430 PRs pending on master.
Thanks Zoltan for this great initiative!
Cheers,
Panagiotis
On Mon, Jun 1, 2020 at 2:33 PM David Mollitor <dam6...@gmail.com>
wrote:
Thanks so much for the work on this.
Just cleaned up mine.
On Sat, May 30, 2020 at 10:16 AM Zoltan Haindrich <k...@rxd.hu>
wrote:
Hey All,
The new test executor will pick up any PR which doesn't yet have
a test
result - now that the patch is on the master; every PR which is
mergeable
with the master branch is
a good candidate - so the right move would be to clean up our PR
backlog.
I would like to ask everyone to look at
https://github.com/apache/hive/pulls
and close some PRs which are already submitted or just leftovers
>from -
primarily I would ask you to look at PRs opened by yourself...
cheers,
Zoltan
--
Zoltán Haindrich