[email protected] has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13733
Change subject: Modify table scan tool to collect and surface errors instead of crashing. ...................................................................... Modify table scan tool to collect and surface errors instead of crashing. Initialized a Status for each thread in StartWork(), passed as out-parameter to ScanTask() or CopyTask(). Changed signature of ScanData(): replaced CHECKs in ScanData() with return of bad Status. ScanTask() or CopyTask() catch bad Statuses from ScanData() in out-parameter back to StartWork(). StartWork() logs and returns the first bad Status from the thread pool. Change-Id: Ic45da537b8bacfa9625010536ea82da9a6e76100 --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/table_scanner.cc M src/kudu/tools/table_scanner.h 3 files changed, 61 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/13733/1 -- To view, visit http://gerrit.cloudera.org:8080/13733 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ic45da537b8bacfa9625010536ea82da9a6e76100 Gerrit-Change-Number: 13733 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <[email protected]>
