Dinesh Bhat has posted comments on this change.
Change subject: [c++compilation] fixed 'unused' warnings
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4503/1/src/kudu/codegen/row_projector.cc
File src/kudu/co
Alexey Serbin has posted comments on this change.
Change subject: [c++compilation] fixed 'unused' warnings
..
Patch Set 1:
(1 comment)
Oops, it seems I pressed 'Submit' instead of reply. Anyway, I'm definitely
open to change
Alexey Serbin has submitted this change and it was merged.
Change subject: [c++compilation] fixed 'unused' warnings
..
[c++compilation] fixed 'unused' warnings
Use DCHECK() instead of DCHECK_NOTNULL() as recommended by
glog/log
Adar Dembo has posted comments on this change.
Change subject: [c++compilation] fixed 'unused' warnings
..
Patch Set 1: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4503/1/src/kudu/codegen/row_projector.cc
Fil
Alexey Serbin has uploaded a new change for review.
http://gerrit.cloudera.org:8080/4503
Change subject: [c++compilation] fixed 'unused' warnings
..
[c++compilation] fixed 'unused' warnings
Use DCHECK() instead of DCHECK_NOTN