dongjoon-hyun opened a new pull request, #45:
URL: https://github.com/apache/spark-connect-swift/pull/45

   ### What changes were proposed in this pull request?
   
   This PR aims to change the visibility of the following.
   - `enum SparkConnectError`
   - `StorageLevel` fields
   
   In addition, this PR changes the following test suite to use only `public` 
APIs of `SparkConnect` by changing the import statement. This will help us 
validate the visibility change easily.
   - `DataFrameReaderTests`
   - `DataFrameTests`
   - `DataFrameWriterTests`
   - `SQLTests`
   
   ### Why are the changes needed?
   
   To allow users to use `SparkConnectError` and `StorageLevel`.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This is a visibility change on the unreleased versions.
   
   ### How was this patch tested?
   
   Pass the CIs and manual review.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No.


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to