[GitHub] [iceberg] massdosage commented on a change in pull request #2394: Doc: Document non-atomicity of Hive multi-table inserts

[GitBox <gi...@apache.org>]

massdosage commented on a change in pull request #2394:
URL: https://github.com/apache/iceberg/pull/2394#discussion_r604782225



##########
File path: site/docs/hive.md
##########
@@ -121,3 +121,11 @@ Columns from the Hive SQL `SELECT` clause are projected down to the Iceberg read
 
 #### Hive Query Engines
 Both the Map Reduce and Tez query execution engines are supported.
+
+#### Hive Multi-table inserts
+It is possible to issue inserts targeting multiple Hive tables backed by Iceberg. The users should be aware that the commits are atomic only on table level, and the commit is not atomic on query level. The commits to Iceberg tables happen 1-by-1, and if one of the commit fails the other writes will not be rolled back. Example:

Review comment:
       ```suggestion
   It is possible to issue inserts targeting multiple Hive tables backed by Iceberg. Users should be aware that the commits are atomic only at the _table_ level and not at the _query_ level. The commits to Iceberg tables happen 1-by-1, and if one of the commits fails the other writes will not be rolled back. Example:
   ```
   
   (or go with @RussellSpitzer's  rewording above)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org