mathiasschw-db opened a new pull request, #25:
URL: https://github.com/apache/spark-connect-go/pull/25

   ### What changes were proposed in this pull request?
   This change pull the session code out of the sql module to separate the two 
layers more explicitly in the codebase
   
   
   ### Why are the changes needed?
   This hardens the code and ensures that we don't have dependencies that 
violate the layered structure that we describe in the README. This is the 
change that we prepared with the introduction of the executor interfaces earlier
   
   
   ### Does this PR introduce _any_ user-facing change?
   The session now has to be build from the session package but the 
functionality is unchanged
   
   
   ### How was this patch tested?
   Ran existing test suite


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