You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org> on 2017/08/01 00:00:01 UTC

[Impala-ASF-CR] IMPALA-1891: Statestore won't send deletions in initial non-delta topic

Impala Public Jenkins has submitted this change and it was merged.

Change subject: IMPALA-1891: Statestore won't send deletions in initial non-delta topic
......................................................................


IMPALA-1891: Statestore won't send deletions in initial non-delta topic

Currently, when a subscriber connects to the statestore, its first
update contains the entire topic including deletions which are useless
because the subscriber has no base to apply those deletions to. This
patch ensures that the deletions are not included in that first update.

Testing:
Modified existing test case to include this check

Change-Id: I3fc525e1f3d960d642fc6356abb75f744cab7c33
Reviewed-on: http://gerrit.cloudera.org:8080/7527
Reviewed-by: Matthew Jacobs <mj...@cloudera.com>
Tested-by: Impala Public Jenkins
---
M be/src/statestore/statestore.cc
M tests/statestore/test_statestore.py
2 files changed, 15 insertions(+), 10 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Matthew Jacobs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3fc525e1f3d960d642fc6356abb75f744cab7c33
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Bikramjeet Vig <bi...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bi...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>