voonhous commented on code in PR #19592:
URL: https://github.com/apache/hudi/pull/19592#discussion_r3920794895


##########
website/docs/clustering.md:
##########
@@ -202,6 +202,28 @@ The available strategies are as follows:
    consistent bucket index and only applicable to the Spark engine. Set 
`hoodie.clustering.execution.strategy.class`
    to 
`org.apache.hudi.client.clustering.run.strategy.SparkConsistentBucketClusteringExecutionStrategy`.
 
+#### Row writer
+
+On Spark, the execution strategies above can rewrite the data either through 
the row writer, which operates on
+a `Dataset<Row>` and avoids converting records to Avro, or through the older 
RDD path. Which one runs is decided
+by a single config:
+
+| Config Name | Default | Description |
+|-------------|---------|-------------|

Review Comment:
   **nit:** On the same row, feel free to ignore: the tables just above 
(Single-Group Clustering, File-Slice Sort Order, Driver-Side Plan Generation) 
wrap key and default in backticks, and the Execution Strategy table right above 
carries `Config Param:` / `Since Version:`. Could this row follow one of those? 
Plain fence rather than a suggestion block, since this thread anchors on the 
separator line:
   
   ```markdown
   | `hoodie.datasource.write.row.writer.enable` | `true` | When enabled, 
clustering rewrites file groups through the Spark row writer instead of the RDD 
path.<br /><br />`Config Param: ENABLE_ROW_WRITER`<br />`Since Version: 0.9.0` |
   ```



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

Reply via email to