adam-christian-software opened a new pull request, #3188:
URL: https://github.com/apache/polaris/pull/3188

   # Context
   This is the same change as https://github.com/apache/polaris/pull/3026 but I 
accidentally destroyed my repo.
   
   This change fixes https://github.com/apache/polaris/issues/3021 by creating 
the Spark 4 client. Most of the changes are just simply copying and pasting 
what already exists and changing to match the proper interfaces.
   
   I will carve out the overlapping business logic into a separate module later.
   
   # Test
   Move over the same testing framework with what exists in the 3.5 plugin. In 
addition, I tested E2E with Parquet.
   
   ## Checklist
   - [X] ๐Ÿ›ก๏ธ Don't disclose security issues! (contact [email protected])
   - [X] ๐Ÿ”— Clearly explained why the changes are needed, or linked related 
issues: Fixes #
   - [X] ๐Ÿงช Added/updated tests with good coverage, or manually tested (and 
explained how)
   - [X] ๐Ÿ’ก Added comments for complex logic
   - [X] ๐Ÿงพ Updated `CHANGELOG.md` (if needed)
   - [BLOCKED] ๐Ÿ“š Updated documentation in `site/content/in-dev/unreleased` (if 
needed). I need the updates in https://github.com/apache/polaris/pull/3000 
first.
   


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