Hello Yuqi Du, Alexey Serbin, Yingchun Lai, Yifan Zhang, Kudu Jenkins, KeDeng,

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

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

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

Change subject: [KUDU-3483] Fixbug of auto flush data when table schema changed
......................................................................

[KUDU-3483] Fixbug of auto flush data when table schema changed

In auto_flush_background mode, applying an operation firstly
inserts the row into the buffer. When the buffer is full or
function flush() is called, it tries to flush multiple data into
Kudu server. Firstly, it groups these data according to the tablet
id as a batch. A batch may contains multiple rows which belong to
the same tablet. Then a batch will encode into bytes. At this time,
it reads the table schema of the first row and decides the format
of the data. If two rows has different schema but belongs to the same
tablet, which maybe because of altering the table between inserting
two rows, it causes array index outbound exception.

This patch will validate the schema of multiple rows which belong
to the same tablet. If the schema is different, it puts them into
the different groups as different batches.

Change-Id: Ie6501962b32814d121f180b2942999c402d927db
---
M java/kudu-client/src/main/java/org/apache/kudu/Schema.java
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
3 files changed, 270 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/19949/8
--
To view, visit http://gerrit.cloudera.org:8080/19949
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie6501962b32814d121f180b2942999c402d927db
Gerrit-Change-Number: 19949
Gerrit-PatchSet: 8
Gerrit-Owner: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com>

Reply via email to