Hello Dan Burkert, Adar Dembo, Alexey Serbin, Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/4518

to look at the new patch set (#2).

Change subject: [misc] : Remove few more warnings
......................................................................

[misc] : Remove few more warnings

Observed few warnings with fresh builds:
[365/1113] Building CXX object 
src/kudu/common/CMakeFiles/kudu_common_exported.dir/row_operations.cc.o
../../src/kudu/common/row_operations.cc: In member function ‘std::string 
kudu::DecodedRowOperation::ToString(const kudu::Schema&) const’:
../../src/kudu/common/row_operations.cc:58:1: warning: control reaches end of 
non-void function [-Wreturn-type]
 }
 ^
[403/1113] Building CXX object 
src/kudu/common/CMakeFiles/kudu_common.dir/row_operations.cc.o
../../src/kudu/common/row_operations.cc: In member function ‘std::string 
kudu::DecodedRowOperation::ToString(const kudu::Schema&) const’:
../../src/kudu/common/row_operations.cc:58:1: warning: control reaches end of 
non-void function [-Wreturn-type]
 }
 ^
[516/1113] Building CXX object src/kudu/benchmarks/CMakeFiles/rle.dir/rle.cc.o
../../src/kudu/benchmarks/rle.cc: In function ‘void kudu::BooleanRLE()’:
../../src/kudu/benchmarks/rle.cc:93:10: warning: variable ‘run_length’ set but 
not used [-Wunused-but-set-variable]
   size_t run_length;
          ^
[575/1113] Building CXX object 
src/kudu/cfile/CMakeFiles/cfile.dir/compression_codec.cc.o
In file included from ../../src/kudu/cfile/compression_codec.h:24:0,
                 from ../../src/kudu/cfile/compression_codec.cc:26:
../../src/kudu/gutil/macros.h:102:0: warning: "DISALLOW_COPY_AND_ASSIGN" 
redefined
 #define DISALLOW_COPY_AND_ASSIGN(TypeName) \
 ^
In file included from ../../thirdparty/installed-deps/include/snappy.h:45:0,
                 from ../../src/kudu/cfile/compression_codec.cc:20:
../../thirdparty/installed-deps/include/snappy-stubs-public.h:79:0: note: this 
is the location of the previous definition
 #define DISALLOW_COPY_AND_ASSIGN(TypeName) \
 ^

Also added a trivial comment update.

Change-Id: I89b96d52dfed6b38f17cf8cdebeed840fb32f98d
---
M src/kudu/benchmarks/rle.cc
M src/kudu/codegen/row_projector.cc
M src/kudu/common/row_operations.cc
M src/kudu/gutil/macros.h
4 files changed, 7 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/4518/2
-- 
To view, visit http://gerrit.cloudera.org:8080/4518
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I89b96d52dfed6b38f17cf8cdebeed840fb32f98d
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to