impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3725 Support Kudu UPSERT in Impala
Date Wed, 19 Oct 2016 23:59:12 GMT
Alex Behm has posted comments on this change.

Change subject: IMPALA-3725 Support Kudu UPSERT in Impala

Patch Set 8:

File fe/src/main/java/org/apache/impala/analysis/

Line 689:       throw new AnalysisException("UPSERT does not currently support any plan hints.");
make this a warning instead like we do for unrecognized hints
File fe/src/test/java/org/apache/impala/analysis/

Line 3456:   public void TestUpsert() {
Let's move this into a new file. This file is already huge enough :)
File fe/src/test/java/org/apache/impala/analysis/

Line 1653:       ParsesOk(String.format("upsert into %s t [shuffle] select a from src", optTbl));
move into TestPlanHints()
File testdata/workloads/functional-planner/queries/PlannerTest/insert.test:

Line 594
> Yeah, I'm not sure I know what you mean here.
Something like this:

upsert into functional_kudu.testtbl
select, v.int_col, c.cnt from (
  select id, int_col, count(*) cnt from
  group by 1, 2) v
where cnt < 10
File testdata/workloads/functional-planner/queries/PlannerTest/kudu-upsert.test:

Line 40:    runtime filters: RF000 -> a.string_col
add DISTRIBUTEDPLAN here also just to make sure we can generate one
File testdata/workloads/functional-query/queries/QueryTest/kudu_crud.test:

Line 289: upsert into table tdata (id, valf) values (2, 10), (120, 20), (0, 0)
throw in a few NULLs somewhere

Line 306: upsert into table tdata (valb, name, id)
add an upsert without a query stmt

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I8df5cea36b642e267f85ff6b163f3dd96b8386e9
Gerrit-PatchSet: 8
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: Thomas Tauber-Marshall <>
Gerrit-HasComments: Yes

View raw message