[kudu-CR] [c++compilation] fixed 'unused' warnings

2016-09-22 Thread Dinesh Bhat (Code Review)
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

[kudu-CR] [c++compilation] fixed 'unused' warnings

2016-09-21 Thread Alexey Serbin (Code Review)
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

[kudu-CR] [c++compilation] fixed 'unused' warnings

2016-09-21 Thread Adar Dembo (Code Review)
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

[kudu-CR] [c++compilation] fixed 'unused' warnings

2016-09-21 Thread Alexey Serbin (Code Review)
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