Skip to content

[NGSOK-1746] Spark 4.2 support#26

Closed
giggsoff wants to merge 3 commits into
arenadata:develop/4.3.0/1.10.1.1from
giggsoff:feature/NGSOK-1746
Closed

[NGSOK-1746] Spark 4.2 support#26
giggsoff wants to merge 3 commits into
arenadata:develop/4.3.0/1.10.1.1from
giggsoff:feature/NGSOK-1746

Conversation

@giggsoff

@giggsoff giggsoff commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator
  1. Copy 4.0 into 4.2
  2. Pull upstream changes for 4.2 support
  3. Align tests with our Spark 4.2

Petr Fedchenkov added 2 commits June 23, 2026 19:03
Adapt the v4.2 tree (copied from v4.0) to the Apache Spark 4.2.0 APIs and
wire the new engine modules into the build:

- Port the Spark 4.2.0 view/geo APIs: ViewCatalog/SupportsReplaceView ->
  TableViewCatalog/ViewInfo, GeographyVal/GeometryVal ->
  BinaryView + Geography/GeometryType, and add reportDriverMetrics() to
  the staged/truncatable tables.
- Build wiring: settings.gradle, gradle.properties (known/default Spark
  4.2), spark/build.gradle, spark/v4.2/build.gradle (4.0->4.2, Scala
  target 17) and the spark-ci matrix.
- Work around the iceberg-arrow netty pin: force netty-buffer/common
  4.2.13 for the v4.2 build only (Spark 4.2 needs
  PlatformDependent.hasDirectByteBufferAddress, added in 4.2.12).
- Update 4.2 test expectations and remaining 4.0->4.2 references.
Downstream-only changes layered on top of the generic Spark 4.2 support:

- Depend on the Arenadata Spark fork artifact 4.2.0.1-4.3.0-1 instead of
  vanilla Apache Spark (gradle/libs.versions.toml).
- Publish the 4.2 spark/extensions/runtime modules to the Arenadata
  GitHub Packages repo (.github/workflows/publish.yml).
- Work around the fork's ALTER TABLE RENAME behavior in the managed-table
  migration tests: a managed source must be marked
  'external.table.purge'='true' for migrateTable to find its data files
  (TestCreateActions#markPurgeableForMigration).
@giggsoff giggsoff force-pushed the feature/NGSOK-1746 branch from e3b25ea to 30ae7bc Compare June 23, 2026 16:04
@giggsoff giggsoff marked this pull request as ready for review June 23, 2026 16:05
@giggsoff

Copy link
Copy Markdown
Collaborator Author

replaced by #27

@giggsoff giggsoff closed this Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant