Wenzhe Zhou has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19651 )

Change subject: IMPALA-11985: [DOCS] Support for Kudu's multi-rows transaction
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/19651/1/docs/topics/impala_kudu.xml
File docs/topics/impala_kudu.xml:

http://gerrit.cloudera.org:8080/#/c/19651/1/docs/topics/impala_kudu.xml@1403
PS1, Line 1403: or per-session
              :         level.
> What does this do on per-session level, i.e. what's the semantics of a tran
When ENABLE_KUDU_TRANSACTION is set as true for an opening Impala session, 
Impala will open separate Kudu transaction for each INSERT or CTAS statement. 
One Kudu transaction cannot cover more than one INSERT/CTAS statements. Need to 
tweak the sentences to avoid confusing.


http://gerrit.cloudera.org:8080/#/c/19651/1/docs/topics/impala_kudu.xml@1424
PS1, Line 1424: When you enable this option
> What option? --kudu_ignore_conflicts_in_transaction or ENABLE_KUDU_TRANSACT
Yes, it's query option


http://gerrit.cloudera.org:8080/#/c/19651/1/docs/topics/impala_kudu.xml@1442
PS1, Line 1442: insert rows into multiple tables
> How is that achievable with current support for multi-row Kudu transactions
This line should be removed.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic226679d83d7221f843994ead11cb2bc9e971882
Gerrit-Change-Number: 19651
Gerrit-PatchSet: 1
Gerrit-Owner: Shajini Thayasingh <sthayasi...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Comment-Date: Fri, 24 Mar 2023 21:48:45 +0000
Gerrit-HasComments: Yes

Reply via email to