Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14021 )

Change subject: IMPALA-8811 IMPALA-8813: [DOCS] Document the ACID support in 
Impala
......................................................................


Patch Set 5:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/14021/6/docs/topics/impala_default_transactional_type.xml
File docs/topics/impala_default_transactional_type.xml:

http://gerrit.cloudera.org:8080/#/c/14021/6/docs/topics/impala_default_transactional_type.xml@58
PS6, Line 58: The
            :           <codeph>INSERT</codeph> operations will be 
transactional on this table.
> I think that the best thing here would be a link to impala_transactions.
Done


http://gerrit.cloudera.org:8080/#/c/14021/5/docs/topics/impala_transactions.xml
File docs/topics/impala_transactions.xml:

http://gerrit.cloudera.org:8080/#/c/14021/5/docs/topics/impala_transactions.xml@35
PS5, Line 35:       satisfy ACID (Atomicity, Consistency, Isolation, 
Durability) properties, which intend to
> I totally agree with Tim, we should think it through carefully what to writ
Rewored more.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0320e573301175dedb00d6e745f9f973519a16bb
Gerrit-Change-Number: 14021
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Greg Rahn <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Tamas Mate <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Sat, 10 Aug 2019 00:07:48 +0000
Gerrit-HasComments: Yes

Reply via email to