andygrove commented on code in PR #4362:
URL: https://github.com/apache/datafusion-comet/pull/4362#discussion_r3263617174
##########
docs/source/user-guide/latest/understanding-comet-plans.md:
##########
@@ -145,13 +145,13 @@ by role. Names match what is shown in the plan output.
### Scans
-| Node | Description
|
-| ------------------------ |
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
|
-| `CometScan` | V1 Parquet scan driven by Spark's file-source
path through Comet's Parquet reader. Decoding runs in native code; the
resulting Arrow batches cross JNI into the native plan. The active scan
implementation is shown in brackets, e.g. `CometScan [native_iceberg_compat]`. |
-| `CometBatchScan` | DataSource V2 scan, including Iceberg Parquet,
that produces Arrow batches consumed by Comet.
|
-| `CometNativeScan` | Fully native Parquet scan that runs entirely in
DataFusion (no JVM Parquet reader involvement).
|
-| `CometIcebergNativeScan` | Fully native Iceberg Parquet scan.
|
-| `CometCsvNativeScan` | Fully native CSV scan (experimental).
|
+| Node | Description
|
+| ------------------------ |
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
|
+| `CometScan` | V1 Parquet scan driven by Spark's file-source
path through Comet's Parquet reader. Decoding runs in native code; the
resulting Arrow batches cross JNI into the native plan. |
+| `CometBatchScan` | DataSource V2 scan, including Iceberg Parquet,
that produces Arrow batches consumed by Comet.
|
+| `CometNativeScan` | Fully native Parquet scan that runs entirely in
DataFusion (no JVM Parquet reader involvement).
|
+| `CometIcebergNativeScan` | Fully native Iceberg Parquet scan.
|
+| `CometCsvNativeScan` | Fully native CSV scan (experimental).
|
Review Comment:
`CometScan` should no longer appear. I will update this.
--
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]
