+1 from me too On Sat, Jul 18, 2015 at 3:32 AM, Ted Yu <[email protected]> wrote: > +1 to removing commit messages. > > > >> On Jul 18, 2015, at 1:35 AM, Sean Owen <[email protected]> wrote: >> >> +1 to removing them. Sometimes there are 50+ commits because people >> have been merging from master into their branch rather than rebasing. >> >>> On Sat, Jul 18, 2015 at 8:48 AM, Reynold Xin <[email protected]> wrote: >>> I took a look at the commit messages in git log -- it looks like the >>> individual commit messages are not that useful to include, but do make the >>> commit messages more verbose. They are usually just a bunch of extremely >>> concise descriptions of "bug fixes", "merges", etc: >>> >>> cb3f12d [xxx] add whitespace >>> 6d874a6 [xxx] support pyspark for yarn-client >>> >>> 89b01f5 [yyy] Update the unit test to add more cases >>> 275d252 [yyy] Address the comments >>> 7cc146d [yyy] Address the comments >>> 2624723 [yyy] Fix rebase conflict >>> 45befaa [yyy] Update the unit test >>> bbc1c9c [yyy] Fix checkpointing doesn't retain driver port issue >>> >>> >>> Anybody against removing those from the merge script so the log looks >>> cleaner? If nobody feels strongly about this, we can just create a JIRA to >>> remove them, and only keep the author names. >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] >
--------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
