Hello! Thanks for bringing this up! As Vihang said, our team created the account `dataproc-metastore` due to the newness of our product and internal contributing challenges we wanted to work out before moving forward. Moving forward we see the benefit of using our personal accounts for attribution and clarity while doing open source work and will migrate to using those.
We'll go ahead and re-open that PR under either Zhou or my account and are excited to be part of this open source community in the future! Cheers, Cameron On 2021/02/04 20:11:26, Vihang Karajgaonkar <v...@apache.org> wrote: > Thanks Zoltan for your email.> > > Just to give some context, dataproc-metastore is Google's metastore> > compatible cloud service. The good news is that they are happy and willing> > to contribute any improvements/fixes to Apache Hive (metastore> > specifically) instead of forking out the repository.> > They also contributed their proposed changes here:> > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=158869886> > > I think it makes sense to have individual users contribute the PR so that> > we can attribute the patch accordingly. When I merged their PR I asked them> > offline who is the end user for this PR and they mentioned they are still> > figuring out who is going to be the point of contact for the open-source> > contributions. While merging the PR, github suggested the author name and I> > used that.> > > I was a bit angry because of the above; so I've closed it.> > >> > I feel this is a bit against the spirit of open-source hive and it would be> > great to have a wiki page for commit guidelines and ask them to refer to> > it. The only wiki that I find about commit guidelines is> > https://cwiki.apache.org/confluence/display/Hive/HowToCommit which> > definitely needs an update.> > > On Thu, Feb 4, 2021 at 1:02 AM Zoltan Haindrich <ki...@rxd.hu> wrote:> > > > Hey All!> > >> > > It seems to me that someone have opened a "dataproc-metastore" account on> > > github and is contributing to Hive thru that user.> > > I personally don't like that the account is not a real person - it looks> > > more like a team or group inside Google.> > >> > > This account already has a commit which is very confusing:> > > * the github account is https://github.com/dataproc-metastore> > > * the jira is assigned to Cameron Moberg> > > https://issues.apache.org/jira/browse/HIVE-24470> > > * the actual commits in the PR were made by Zhou Fang> > > https://github.com/coufon> > > * the commit is attributed to "Zhou Fang" -> > > https://github.com/apache/hive/commit/b0309b7f023d9785c3a842d70d0fc471252101bf> > > * the jira is still open...but that's not really relevant - that can be> > > fixed in no time :D> > >> > > I think we should stop merging PRs from sources like this (or is it too> > > much to ask that the user should have a matching github account)?> > >> > > This "dataproc-metastore" user had one more PR open - I was a bit angry> > > because of the above; so I've closed it.> > >> > > Let me know what you think!> > >> > > cheers,> > > Zoltan> > >> >
