RussellSpitzer commented on a change in pull request #4325:
URL: https://github.com/apache/iceberg/pull/4325#discussion_r834683946



##########
File path: docs/spark/spark-procedures.md
##########
@@ -417,10 +417,11 @@ To leave the original table intact while testing, use 
[`snapshot`](#snapshot) to
 
 #### Usage
 
-| Argument Name | Required? | Type | Description |
-|---------------|-----------|------|-------------|
-| `table`       | ✔️  | string | Name of the table to migrate |
-| `properties`  | ️   | map<string, string> | Properties for the new Iceberg 
table |
+| Argument Name        | Required? | Type                | Description         
                           |
+|----------------------|-----------|---------------------|------------------------------------------------|
+| `table`              | ✔️        | string              | Name of the table 
to migrate                   |
+| `properties`         | ️         | map<string, string> | Properties for the 
new Iceberg table           |
+| `skip_corrupt_files` | ️         | boolean             | If true, skip 
corrupt files (false by default) |

Review comment:
       I think we need a better term than "corrupt" here. Maybe "skip on 
error"? Then the description is "If true, skips files which cannot be imported 
into Iceberg"?




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