Oops. I was sync'ing gerrit/cdh5-trunk and asf/master. They became off-sync after the move to asf. Shouldn't this be cherry-picked ?
On Tue, Aug 2, 2016 at 5:27 PM, Lars Volker <[email protected]> wrote: > 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]> >> > -- Thanks, Bharath
