jiangxt2 opened a new pull request, #13149:
URL: https://github.com/apache/gravitino/pull/13149

   ### What changes were proposed in this pull request?
   
   Add an end-to-end Spark integration test that reads a historical Lance 
snapshot through the Gravitino Lance REST catalog. The test uses a dedicated 
`lance_history` Spark catalog and Gravitino catalog configured with 
`VERSION_CHECK`, writes two Lance versions, verifies the latest snapshot, and 
verifies that `VERSION AS OF` returns only the earlier rows.
   
   ### Why are the changes needed?
   
   The existing Lance Spark REST integration tests cover current-version reads 
but do not prove that Spark time-travel reads remain correct when table 
metadata is resolved through Gravitino. This regression test validates the 
existing separation between Gravitino table metadata resolution and Lance Spark 
snapshot selection across the declared Lance Spark versions 0.2.0, 0.4.0, and 
0.5.1. No Issue is referenced because this is a standalone test contribution 
for an existing capability, not a production bug fix.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This change only adds integration-test coverage and does not modify 
production code, REST behavior, or `describeTable(version)` metadata semantics.
   
   ### How was this patch tested?
   
   Ran `./gradlew :lance:lance-rest-server:lanceSparkMatrixTest 
-PlanceSparkBundleVersions=0.2.0,0.4.0,0.5.1 -PtestMode=embedded 
-PskipDockerTests=true`.
   
   The matrix completed with 13 tests per bundle, 0 skipped, 0 failures, and 0 
errors. The new historical-read test passed for all three Lance Spark bundle 
versions.
   


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

Reply via email to