Hi Bharath, Did you mean to merge this? Or did I misinterpret the Gerrit mail?
Cheers, Lars On Aug 2, 2016 12:55, "Bharath Vissapragada (Code Review)" < [email protected]> wrote: > Bharath Vissapragada has submitted this change and it was merged. > > Change subject: IMPALA-2979: Fix scheduling on remote hosts > ...................................................................... > > > IMPALA-2979: Fix scheduling on remote hosts > > Also fixes: IMPALA-2400, IMPALA-3043 > > This change fixes scheduling scan-ranges on remote hosts by adding > remote backend selection capability to SimpleScheduler. Prior to this > change the scheduler would try to select a local backend even when > remote scheduling was requested. > > This change also allows pseudo-randomized remote backend selection to > prevent convoying, which could happen when different independent > schedulers had the same internal state, e.g. after a cluster restart. To > enable the new behavior set the query option SCHEDULE_RANDOM_REPLICA to > true. > > This change also fixes IMPALA-2400: Unpredictable locality behavior > for reading Parquet files > > This change also fixes IMPALA-3043: SimpleScheduler does not handle > hosts with multiple IP addresses correctly > > This change also does some clean-up in scheduler.h and > simple-scheduler.{h,cc}. > > Change-Id: I044f83806fcde820fcb38047cf6b8e780d803858 > Reviewed-on: http://gerrit.cloudera.org:8080/3771 > Reviewed-by: Lars Volker <[email protected]> > Reviewed-by: Sailesh Mukil <[email protected]> > Tested-by: Internal Jenkins > (cherry picked from commit 0ad935b63c23029bd4cac4beefde6b3b7c0e322b) > --- > M be/src/scheduling/scheduler.h > M be/src/scheduling/simple-scheduler-test.cc > M be/src/scheduling/simple-scheduler.cc > M be/src/scheduling/simple-scheduler.h > M be/src/service/query-options.cc > M be/src/service/query-options.h > M common/thrift/ImpalaInternalService.thrift > M common/thrift/ImpalaService.thrift > M fe/src/main/java/com/cloudera/impala/analysis/TableRef.java > M fe/src/test/java/com/cloudera/impala/analysis/AnalyzeStmtsTest.java > M fe/src/test/java/com/cloudera/impala/analysis/ParserTest.java > M fe/src/test/java/com/cloudera/impala/analysis/ToSqlTest.java > 12 files changed, 984 insertions(+), 489 deletions(-) > > > > -- > To view, visit http://gerrit.cloudera.org:8080/2200 > To unsubscribe, visit http://gerrit.cloudera.org:8080/settings > > Gerrit-MessageType: merged > Gerrit-Change-Id: I044f83806fcde820fcb38047cf6b8e780d803858 > Gerrit-PatchSet: 33 > Gerrit-Project: Impala > Gerrit-Branch: cdh5-trunk > Gerrit-Owner: Lars Volker <[email protected]> > Gerrit-Reviewer: Bharath Vissapragada <[email protected]> > Gerrit-Reviewer: Internal Jenkins > Gerrit-Reviewer: Lars Volker <[email protected]> > Gerrit-Reviewer: Marcel Kornacker <[email protected]> > Gerrit-Reviewer: Matthew Jacobs <[email protected]> > Gerrit-Reviewer: Mostafa Mokhtar <[email protected]> > Gerrit-Reviewer: Sailesh Mukil <[email protected]> > Gerrit-Reviewer: anujphadke <[email protected]> >
