[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4985 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. IMPALA-3710: Kudu DML should ignore conflicts, pt2 Second part of IMPALA-3710, which removed

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4985 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#3). Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. IMPALA-3710: Kudu DML should ignore conflicts, pt2 Second part of IMPALA-3710, which removed the IGNORE DML

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4985/2/be/src/exec/kudu-table-sink.cc File

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. IMPALA-3710: Kudu DML should ignore conflicts, pt2 Second part of IMPALA-3710, which removed the IGNORE DML

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 1: BTW, based on a conversation with the Kudu folks I learned my handling of the 'NotFound'

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4985/1/be/src/exec/data-sink.cc File

[Impala-ASF-CR] IMPALA-3710: Kudu DML should ignore conflicts, pt2

2016-11-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4985/1/be/src/exec/data-sink.cc File