Repository navigation
Add support for InMemoryRelation #137
Description
What is the problem the feature request solves?
Currently Comet cannot be triggered if Spark users read data from cached RDD. To support this use case, we'll need to add support for Spark's InMemoryRelation.
It looks like we may need to implement Arrow for CachedBatchSerializer.
Describe the potential solution
Add Comet support for InMemoryRelation, so that Spark query starts from cached RDD can also use Comet native execution.
Additional context
It is not a priority as of now, but will be something good to have in future.
Activity
advancedxy commentedon Mar 1, 2024
Another way to read
InMemoryRelationis to wrapped it with anCometRowToColumnarExeclike I proposed in #119sunchao commentedon Mar 1, 2024
@advancedxy Yea,
CometRowToColumnarExeccould be a more general solution, not only forInMemoryRelation, but also for other types of data sources like CSV, JSON, etc. The advantage of implementing Arrow forCachedBatchSerializerhere is that we can avoid the extra cost from row to columnar conversion, and potentially be more space efficient because of better compression.advancedxy commentedon Mar 1, 2024
Yea, of course. I can get the rational. We could always add specialized operators to improve performance as long as it's worth the effort and there's interest to implement it.
perf(contrib-delta) P7m: O(1) partition-value lookup in build_delta_p…