Skip to content

Conversation

@varun-edachali-dbx
Copy link
Contributor

@varun-edachali-dbx varun-edachali-dbx commented May 20, 2025

What type of PR is this?

  • Refactor

Description

Separate the Session related functionality from the Connection class, in order to:

  • Better align the object structure with that of the JDBC driver
  • Further abstract the backend implementation details from the connection, to eventually give way to the introduction of SEA.

How is this tested?

  • Unit tests
    • some of the existing unit tests were slightly altered to account for the introduction of the Session class. No unit tests were removed or introduced.
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

https://docs.google.com/document/d/1Y-eXLhNqqhrMVGnOlG8sdFrCxBTN1GdQvuKG4IfHmo0/edit?usp=sharing

ensure maintenance of current APIs of Connection while delegating
responsibility

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
… through Connection

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
as in CONTRIBUTING.md

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant