Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12299 )
Change subject: IMPALA-2990: timeout unresponsive queries in coordinator ...................................................................... Patch Set 10: Code-Review+2 (5 comments) http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/runtime/query-state.cc File be/src/runtime/query-state.cc: http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/runtime/query-state.cc@a444 PS10, Line 444: May be nice to keep this part but I guess it's not as important now that IMPALA-6664 is fixed. http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/service/impala-server.cc@215 PS10, Line 215: DEFINE_int32(status_report_interval_ms, 5000, "Interval between profile reports in " (Advanced) Interval ... http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/service/impala-server.cc@218 PS10, Line 218: DEFINE_int32(status_report_max_retry_s, 600, "Max amount of time in seconds for a " (Advanced) Max ... http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/service/impala-server.cc@220 PS10, Line 220: "--status_report_interval_ms."); Effective only if status_report_interval_ms > 0 http://gerrit.cloudera.org:8080/#/c/12299/10/be/src/service/impala-server.cc@221 PS10, Line 221: DEFINE_int32(status_report_cancellation_padding, 20, "The coordinator will wait " (Advanced) The .. -- To view, visit http://gerrit.cloudera.org:8080/12299 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I196c8c6a5633b1960e2c3a3884777be9b3824987 Gerrit-Change-Number: 12299 Gerrit-PatchSet: 10 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Fri, 26 Apr 2019 21:47:34 +0000 Gerrit-HasComments: Yes
