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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to