comphead commented on code in PR #5634:
URL: https://github.com/apache/datafusion-comet/pull/5634#discussion_r4231938651


##########
docs/source/user-guide/latest/operators.md:
##########
@@ -66,7 +66,7 @@ omitted from the tables below and may be reconsidered based 
on demand:
 | `LocalTableScanExec`    | ⚠️     | Disabled by default; there is no 
acceleration advantage and this operator is typically only used in test code. 
Can be opted into via config 
([#4393](https://github.com/apache/datafusion-comet/pull/4393)).                
                                                                                
                                                                                
                       |
 | `EmptyRelationExec`     | ✅     | Spark 4.0 and later. See [Empty 
Relations](compatibility/operators.md#empty-relations) for native-input support 
and writer fallback.                                                            
                                                                                
                                                                                
                                                   |
 | `RangeExec`             | ⚠️     | Disabled by default. Set 
`spark.comet.exec.range.enabled=true` to generate the rows of `spark.range` and 
SQL `range()` in native code, so the operators above them run natively. It can 
be slower than Spark when those operators are only cheap expressions, such as a 
filter, which Spark compiles together with the range into one loop. Ranges 
whose arithmetic overflows the `Long` range fall back to Spark. |
-| `InMemoryTableScanExec` | ⚠️     | Experimental, disabled by default. Set 
`spark.comet.exec.inMemoryCache.enabled=true` before the application starts so 
Comet installs its Arrow cache serializer. Relations with unsupported column 
types stay in Spark's cache format and fall back. See [In-Memory 
Cache](in-memory-cache.md).                                                     
                                                               |
+| `InMemoryTableScanExec` | ⚠️     | Experimental, enabled by default from 
Spark 3.5. Comet installs its Arrow cache serializer as the application starts 
if `spark.comet.exec.inMemoryCache.enabled` is true, its default except on 
Spark 3.4. Relations with unsupported column types stay in Spark's cache format 
and fall back. See [In-Memory Cache](in-memory-cache.md).                       
                                                   |

Review Comment:
   we should remove experimental, or we waiting for more experiments?



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