Skip to content

Enable spark.comet.sparkToColumnar.enabled when running Spark SQL tests #2723

Description

@andygrove

What is the problem the feature request solves?

Many (most?) Spark SQL tests do not start with Parquet scan, but use RDDScanExec, LocalTableScanExec, etc. so Comet falls back to Spark very early. This means that we are missing out on test coverage.

Describe the potential solution

  • Make spark.comet.sparkToColumnar.enabled configurable via env var
  • Set that env var in CI when running Spark SQL tests
  • Fix failing tests (or ignore them until we can fix the bugs)

Additional context

No response

Activity

  1. self-assigned this
    on Nov 6, 2025
  2. modified the milestones: 0.12.0, 0.13.0 on Nov 6, 2025
  3. modified the milestones: 0.13.0, 0.14.0 on Jan 30, 2026
  4. removed this from the 0.14.0 milestone on Mar 12, 2026
  5. added this to the 1.0.0 milestone on May 18, 2026
  6. removed this from the 1.0.0 milestone on Jun 25, 2026
  7. andygrove commented on Jun 25, 2026

    @andygrove
    MemberAuthor

    closing in favor of #4347

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions