hawq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ed Espino <eesp...@pivotal.io>
Subject Re: About the commit
Date Wed, 02 Nov 2016 17:23:29 GMT
Going forward, I will follow the proper dev process as documented here:
https://cwiki.apache.org/confluence/display/HAWQ/Contributing+to+HAWQ

-=e
On Tue, Nov 1, 2016 at 7:43 PM, Hong Wu <xunzhangthu@gmail.com> wrote:

> Hi HAWQ developers,
>
> I found again three invalid commits yesterday in apache HAWQ repo:
>
> 9ccac44a293da650bb5610fa0895b7dc5b35e944
> <https://github.com/apache/incubator-hawq/commit/
> 9ccac44a293da650bb5610fa0895b7dc5b35e944>
> from
> shivzone
> 1e5e996ec9ac8a1fdff6e6797dc9cbf9dd41768c
> <https://github.com/apache/incubator-hawq/commit/
> 1e5e996ec9ac8a1fdff6e6797dc9cbf9dd41768c>
> from
> shivzone
> af656af5951cc76601984e5608939252e4b245ee
> <https://github.com/apache/incubator-hawq/commit/
> af656af5951cc76601984e5608939252e4b245ee>
> from
> Ed
>
> Also, I didn't find any relevant pull requests and JIRAs, it is apache
> community, please keep the commit information clean and follow the apache
> develop process. Next time I find this invalid commit, I think I should do
> revert.
>
> Thanks.
> Hong
>
>
> 2016-06-25 9:30 GMT+08:00 hong wu <xunzhangthu@gmail.com>:
>
> > Hi HAWQ committers,
> >
> > I notice that there are some informal commits of recent check-in. For
> > example:
> > ce3f7c6b5c0315b97298d651f5d5f7383000491a
> > <https://github.com/apache/incubator-hawq/commit/
> ce3f7c6b5c0315b97298d651f5d5f7383000491a>
> > 4d44097085fd139002a255b1032082dc0b030414
> > <https://github.com/apache/incubator-hawq/commit/
> 4d44097085fd139002a255b1032082dc0b030414>
> > 817249a4605abd4415fc0de8e6a545bf88d2aa2e
> > <https://github.com/apache/incubator-hawq/commit/
> 817249a4605abd4415fc0de8e6a545bf88d2aa2e>
> > ...
> >
> > I think we'd better ensure a commit info starting with `HAWQ-#JIRA`. In
> > the following, I try to list the senses developer should pay attention
> to:
> > 1. To avoid commits generated automatically(for example
> > ce3f7c6b5c0315b97298d651f5d5f7383000491a
> > <https://github.com/apache/incubator-hawq/commit/
> ce3f7c6b5c0315b97298d651f5d5f7383000491a>),
> > do not merge or pull from upstream after local commits. Sync with
> upstream
> > master before your local commits.
> > 2. Check commits info before pushing into master. If you found a commit
> > with empty code change, rebase then squash it.
> > 3. If a pull request contains some informal commits(maybe for code review
> > convenience), committers should rebase then squash this temporary commits
> > before pushing into master.
> > 4. If you local develop branch is behind of upstream, it's ok. There are
> > some acceptable reasons for that. For example, during the discuss of your
> > pull request, the master branch is updated. But in this case, the asfgit
> > <https://github.com/asfgit> could not resolve it which means developers
> > should close your pull request manually. So I recommend to attach the
> pull
> > request number in your commits comments(For example
> a57cc9523f97e471a69b65
> > 8556c989d13ad88661
> > <https://github.com/apache/incubator-hawq/commit/
> a57cc9523f97e471a69b658556c989d13ad88661>
> > ).
> >
> > Best
> > xunzhang
> >
> >
>



-- 
*Ed **Espino*
*Cell: *925.389.4640
*WeChat ID:* EdEspino *(*艾德*) *

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message