Zoltan Borok-Nagy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13809 )

Change subject: IMPALA-8710: Increase allowed bit width to 64 for bit packing
......................................................................


Patch Set 5: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13809/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/13809/4//COMMIT_MSG@9
PS4, Line 9: Fixed the buffer overflow that the previous attempt (Change-Id:
> I linked the previous attempt with its change id. Should I also include the
Instead of the change-id I think it's better to link the commit hash or maybe 
the URL to the code review.



--
To view, visit http://gerrit.cloudera.org:8080/13809
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib9276ca291e3d36db16f63f8caf3e9248a18d85b
Gerrit-Change-Number: 13809
Gerrit-PatchSet: 5
Gerrit-Owner: Daniel Becker <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 17 Jul 2019 12:49:29 +0000
Gerrit-HasComments: Yes

Reply via email to