+1 to removing commit messages. 


> On Jul 18, 2015, at 1:35 AM, Sean Owen <so...@cloudera.com> 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 <r...@databricks.com> 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: dev-unsubscr...@spark.apache.org
> For additional commands, e-mail: dev-h...@spark.apache.org
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@spark.apache.org
For additional commands, e-mail: dev-h...@spark.apache.org

Reply via email to