asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: This change includes the following fixes:
Date Wed, 25 Nov 2015 05:53:54 GMT
Till Westmann has posted comments on this change.

Change subject: This change includes the following fixes:
......................................................................


Patch Set 23: Code-Review+2

(1 comment)

Just one comment on the commit message ..

https://asterix-gerrit.ics.uci.edu/#/c/325/23//COMMIT_MSG
Commit Message:

Line 7: This change includes the following fixes:
Could you put a summary of the changes into the 50 chars of this line?
If one uses e.g. "git log --pretty=oneline" this is the only line that's displayed and it
would be nice to get a hint what the change is about :)


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/325
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d167b64bf9ec754182b5b2fe44dfc7e5908c686
Gerrit-PatchSet: 23
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <buyingyi@gmail.com>
Gerrit-Reviewer: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Jianfeng Jia <jianfeng.jia@gmail.com>
Gerrit-Reviewer: Murtadha Hubail <hubailmor@gmail.com>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>
Gerrit-HasComments: Yes

Mime
View raw message