Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17062 )

Change subject: IMPALA-10504: Add tracing for remote block reads
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/17062/2/be/src/runtime/io/hdfs-file-reader.cc
File be/src/runtime/io/hdfs-file-reader.cc:

http://gerrit.cloudera.org:8080/#/c/17062/2/be/src/runtime/io/hdfs-file-reader.cc@81
PS2, Line 81: void appendHostList(const hdfsFS hdfs_fs, const char* file_name, 
int64_t file_offset, int64_t bytes_to_read, ostream& ostr) {
line too long (125 > 90)


http://gerrit.cloudera.org:8080/#/c/17062/2/be/src/runtime/io/hdfs-file-reader.cc@236
PS2, Line 236:         appendHostList(hdfs_fs_, 
scan_range_->file_string()->c_str(), file_offset, bytes_to_read,
line too long (97 > 90)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8c6a3e92f44813048022edf2b91299b3b0a20257
Gerrit-Change-Number: 17062
Gerrit-PatchSet: 2
Gerrit-Owner: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 12 Feb 2021 17:29:08 +0000
Gerrit-HasComments: Yes

Reply via email to