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 <[email protected]>: > 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 > >
