You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org> on 2016/09/01 19:44:57 UTC

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id 

Hello Marcel Kornacker,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/4254

to review the following change.

Change subject: IMPALA-3988: Only use first 96 bits of query id
......................................................................

IMPALA-3988: Only use first 96 bits of query id

This adds utility functions in uid-util.h to create query and instance
ids and convert between them. It also adapts SimpleScheduler to
utilize those functions when creating the instance id
(TPlanFragmentInstanceCtx.fragment_instance_id).

Change-Id: Ia56a03ed9a1d7e77c72b66a01cd48c5b6bf3624f
Reviewed-on: http://gerrit.cloudera.org:8080/4065
Reviewed-by: Marcel Kornacker <ma...@cloudera.com>
Tested-by: Marcel Kornacker <ma...@cloudera.com>
(cherry picked from commit b69e469e99e4b0c08bc4718eb55b041cf7c11ffe)
---
M be/src/scheduling/simple-scheduler.cc
M be/src/service/impala-server.cc
M be/src/util/CMakeLists.txt
A be/src/util/uid-util-test.cc
M be/src/util/uid-util.h
M common/thrift/ImpalaInternalService.thrift
6 files changed, 96 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/54/4254/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4254
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia56a03ed9a1d7e77c72b66a01cd48c5b6bf3624f
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <tm...@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <ma...@cloudera.com>
Gerrit-Reviewer: anujphadke <ap...@cloudera.com>

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id

Posted by "Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org>.
Thomas Tauber-Marshall has abandoned this change.

Change subject: IMPALA-3988: Only use first 96 bits of query id
......................................................................


Abandoned

-- 
To view, visit http://gerrit.cloudera.org:8080/4254
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: Ia56a03ed9a1d7e77c72b66a01cd48c5b6bf3624f
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <tm...@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <ma...@cloudera.com>
Gerrit-Reviewer: anujphadke <ap...@cloudera.com>