Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10573 )
Change subject: IMPALA-7078: Part 2: reduce queue size based on num_scanner_threads ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/10573/2/be/src/exec/hdfs-scan-node.cc File be/src/exec/hdfs-scan-node.cc: http://gerrit.cloudera.org:8080/#/c/10573/2/be/src/exec/hdfs-scan-node.cc@204 PS2, Line 204: f (file_descs_.empty() || progress_.done()) return Status::OK(); > is this somehow related to this change? why do this? This was an error in separating out this from the previous patchset. The previous patchset erroneously deleted this line. -- To view, visit http://gerrit.cloudera.org:8080/10573 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8c0bd14e6bcd9fc1655e344a5307ea0eb4600e8b Gerrit-Change-Number: 10573 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 06 Jun 2018 00:31:25 +0000 Gerrit-HasComments: Yes
