Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 )
Change subject: IMPALA-1048: show sinks in exec summary ...................................................................... Patch Set 12: (3 comments) http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/hbase-table-sink.h File be/src/exec/hbase-table-sink.h: http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/hbase-table-sink.h@38 PS12, Line 38: HBaseTableSink(TDataSinkId sink_id, const RowDescriptor* row_desc, const TDataSink& tsink, line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/hdfs-table-sink.h File be/src/exec/hdfs-table-sink.h: http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/hdfs-table-sink.h@130 PS12, Line 130: HdfsTableSink(TDataSinkId sink_id, const RowDescriptor* row_desc, const TDataSink& tsink, line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/kudu-table-sink.h File be/src/exec/kudu-table-sink.h: http://gerrit.cloudera.org:8080/#/c/11967/12/be/src/exec/kudu-table-sink.h@56 PS12, Line 56: KuduTableSink(TDataSinkId sink_id, const RowDescriptor* row_desc, const TDataSink& tsink, line too long (91 > 90) -- To view, visit http://gerrit.cloudera.org:8080/11967 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3fdf7bacae8ff597b255da65af453e174ba53544 Gerrit-Change-Number: 11967 Gerrit-PatchSet: 12 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Tue, 04 Dec 2018 03:49:14 +0000 Gerrit-HasComments: Yes