szehon-ho commented on code in PR #10644:
URL: https://github.com/apache/iceberg/pull/10644#discussion_r1674784383


##########
docs/docs/spark-writes.md:
##########
@@ -332,6 +332,30 @@ The writer must enable the `mergeSchema` option.
 ```scala
 data.writeTo("prod.db.sample").option("mergeSchema","true").append()
 ```
+
+### Writing to Branches
+The branch must exist before performing write. The operation does **not** 
create the branch if it does not exist.

Review Comment:
   Initially, the "Writing to Branches" seemed a first level header (along with 
"Writing with SQL" and "Writing with Dataframes"), was it intentional?  Do you 
think we can just add an example using Dataframe here?
   
   My thought is, then we dont have to repeat this block of text, which we will 
have to do if we downgrade it to subsections within "Writing with SQL" and 
"Writing with Dataframes".



-- 
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.

To unsubscribe, e-mail: [email protected]

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