Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/13992 )
Change subject: IMPALA-8791: Handle the case where there is no fragment scheduled on the coordinator ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/13992/2/be/src/scheduling/query-schedule.h File be/src/scheduling/query-schedule.h: http://gerrit.cloudera.org:8080/#/c/13992/2/be/src/scheduling/query-schedule.h@81 PS2, Line 81: // Indicates whether this backend will run on the coordinator backend. > nit: wording is a bit weird. Maybe just "Indicates whether this backend is Done http://gerrit.cloudera.org:8080/#/c/13992/2/be/src/scheduling/query-schedule.h@374 PS2, Line 374: requiresCoordinatorFragment > nit: RequiresCoordinatorFragment() Done http://gerrit.cloudera.org:8080/#/c/13992/2/be/src/scheduling/query-schedule.cc File be/src/scheduling/query-schedule.cc: http://gerrit.cloudera.org:8080/#/c/13992/2/be/src/scheduling/query-schedule.cc@261 PS2, Line 261: only_coord_fragment_schduled > scheduled? Done -- To view, visit http://gerrit.cloudera.org:8080/13992 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If5631fa1490d9612ffac3c4c4715348de47d6df2 Gerrit-Change-Number: 13992 Gerrit-PatchSet: 2 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Thu, 08 Aug 2019 18:31:59 +0000 Gerrit-HasComments: Yes
