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 4:

(7 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@1401
PS1, Line 1401: You can control this multi-row transaction feature by using the 
following query option. You
              :         may set this option at per-query or per-se
> This sentence looks like a marketing piece, IMO.  I don't see what useful i
It was removed. Done


http://gerrit.cloudera.org:8080/#/c/19651/1/docs/topics/impala_kudu.xml@1403
PS1, Line 1403: tement.</p>
              :       <codeblo
> When ENABLE_KUDU_TRANSACTION is set as true for an opening Impala session,
Rephrased


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

http://gerrit.cloudera.org:8080/#/c/19651/3/docs/topics/impala_kudu.xml@1438
PS3, Line 1438: With multi-row transaction, you can ato
> With multi-row transaction, you can atomically ingest large number of rows
done


http://gerrit.cloudera.org:8080/#/c/19651/3/docs/topics/impala_kudu.xml@1439
PS3, Line 1439:     ingest large number of rows into a Kudu table with 
INSERT-SELECT or CTAS statement.</p></conbody>
              :   </concept>
              :   <concept id="limitation">
              :     <title>Li
> remove this section
done


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

http://gerrit.cloudera.org:8080/#/c/19651/4/docs/topics/impala_kudu.xml@1423
PS4, Line 1423: impala
Impala


http://gerrit.cloudera.org:8080/#/c/19651/4/docs/topics/impala_kudu.xml@1424
PS4, Line 1424: as part of a new transaction
"with a new opened transaction"


http://gerrit.cloudera.org:8080/#/c/19651/4/docs/topics/impala_kudu.xml@1445
PS4, Line 1445: cannot be a part of a
are not supported to



--
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: 4
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: Shajini Thayasingh <sthayasi...@cloudera.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Comment-Date: Thu, 30 Mar 2023 21:35:33 +0000
Gerrit-HasComments: Yes

Reply via email to