[SPARK-56643][CONNECT][TESTS] Add DSv2 temp view with stored plan tests using Spark Connect mode#55571

Open

longvu-db wants to merge 29 commits into

apache:masterfrom

longvu-db:dsv2-connect-pr1-temp-views-clone

Commits on May 25, 2026

[SPARK-56643][CONNECT][TESTS] Add DSv2 temp view with stored plan tests using Spark Connect mode
longvu-db
committed
Add schema verification assertions for empty table results
longvu-db
committed
Add Scenario 3.1 session column removal test
longvu-db
committed
Empty commit to retrigger CI
longvu-db
committed
Address review: add helpers, fix assertRows, generic serverCatalog
longvu-db
committed
Add session and cache variants for Scenario 7, named args, clearCachingCatalog
longvu-db
committed
Format withTableAndView calls: one argument per line
longvu-db
committed
Remove unnecessary ClassTag from serverCatalog helper
longvu-db
committed
Add externalAppend helper to reduce boilerplate in connect tests
longvu-db
committed
Remove trailing empty line
longvu-db
committed
Rename withTableAndView to withTableAndViews
longvu-db
committed
Fix scalafmt formatting in DataSourceV2TempViewConnectSuite
longvu-db
committed
Address review: add row= named arg, document cache and schemaAfterDrops design
longvu-db
committed
Remove CachingInMemoryTableCatalog.scala and InMemoryTableCatalog.scala from this PR (already in master)
longvu-db
committed
Restore catalog files to master version
longvu-db
committed
Fix compilation error: call clearCache() on catalog instance, not companion object
longvu-db
committed
Address review comments from cloud-fan
longvu-db
committed
Refactor: share temp view tests between classic and Connect via trait
longvu-db
committed
Fix self-review findings: remove missing trait mixin, dead imports, broken Scaladoc
longvu-db
committed
Fix scalafmt formatting in DataSourceV2TempViewConnectSuite
longvu-db
committed
Add comment explaining why sameRows is used instead of checkAnswer for Connect
longvu-db
committed
Address auto-review findings: narrow base trait, clarify Connect comments
longvu-db
committed
Clean up comments and naming: rename cat to catalog, simplify Scaladoc
longvu-db
committed
Make testPrefix abstract, require explicit prefix in each suite
longvu-db
committed
Simplify withTestTableAndViews Scaladoc: remove implementation details
longvu-db
committed
Fully qualify cross-package Scaladoc link for DataSourceV2TempViewConnectSuite
longvu-db
committed
Remove redundant mixin list from DSv2TempViewWithStoredPlanTests Scaladoc
longvu-db
committed
Address review: preserve classic test names, add ClassTag require check
longvu-db
committed
Simplify testPrefix Scaladoc
longvu-db
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[SPARK-56643][CONNECT][TESTS] Add DSv2 temp view with stored plan tests using Spark Connect mode#55571

[SPARK-56643][CONNECT][TESTS] Add DSv2 temp view with stored plan tests using Spark Connect mode#55571
longvu-db wants to merge 29 commits into
apache:masterfrom
longvu-db:dsv2-connect-pr1-temp-views-clone

Commits on May 25, 2026

[SPARK-56643][CONNECT][TESTS] Add DSv2 temp view with stored plan tests using Spark Connect mode

Add schema verification assertions for empty table results

Add Scenario 3.1 session column removal test

Empty commit to retrigger CI

Address review: add helpers, fix assertRows, generic serverCatalog

Add session and cache variants for Scenario 7, named args, clearCachingCatalog

Format withTableAndView calls: one argument per line

Remove unnecessary ClassTag from serverCatalog helper

Add externalAppend helper to reduce boilerplate in connect tests

Remove trailing empty line

Rename withTableAndView to withTableAndViews

Fix scalafmt formatting in DataSourceV2TempViewConnectSuite

Address review: add row= named arg, document cache and schemaAfterDrops design

Remove CachingInMemoryTableCatalog.scala and InMemoryTableCatalog.scala from this PR (already in master)

Restore catalog files to master version

Fix compilation error: call clearCache() on catalog instance, not companion object

Address review comments from cloud-fan

Refactor: share temp view tests between classic and Connect via trait

Fix self-review findings: remove missing trait mixin, dead imports, broken Scaladoc

Fix scalafmt formatting in DataSourceV2TempViewConnectSuite

Add comment explaining why sameRows is used instead of checkAnswer for Connect

Address auto-review findings: narrow base trait, clarify Connect comments

Clean up comments and naming: rename cat to catalog, simplify Scaladoc

Make testPrefix abstract, require explicit prefix in each suite

Simplify withTestTableAndViews Scaladoc: remove implementation details

Fully qualify cross-package Scaladoc link for DataSourceV2TempViewConnectSuite

Remove redundant mixin list from DSv2TempViewWithStoredPlanTests Scaladoc

Address review: preserve classic test names, add ClassTag require check

Simplify testPrefix Scaladoc