Hello Zoltan Chovan, Alexey Serbin, Yingchun Lai, Yifan Zhang,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: KUDU-3518: Fix the unexpected scan predicate
......................................................................

KUDU-3518: Fix the unexpected scan predicate

With the changes introduced in a previous patch[1], there has been
reports of errors during scan operations. This patches fixes these
scan errors.

The issue happens when the equality predicate on a primary key
column is a string and has extra NULL characters appended to the
end of upper bound of the ScanSpec resulting in the addition of
a new predicate (IS NOT NULL) on the next primary key column (if
it exists) and causing the scan failure. This occurs when a
specific code path is taken which needs compactions/flushes to
have taken place complicating the regular C++ tests.

There will be follow-up patches to include tests of this failure.

[1] https://gerrit.cloudera.org/#/c/18434/

Change-Id: I220df7ec1b4f95726c913a19125f4647267d12b1
---
M src/kudu/tablet/cfile_set.cc
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/44/21244/2
--
To view, visit http://gerrit.cloudera.org:8080/21244
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I220df7ec1b4f95726c913a19125f4647267d12b1
Gerrit-Change-Number: 21244
Gerrit-PatchSet: 2
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to