Skip to content

Conversation

varun-edachali-dbx
Copy link
Contributor

@varun-edachali-dbx varun-edachali-dbx commented Jul 7, 2025

What type of PR is this?

  • Refactor

Description

  • Remove Client param from ResultSet, infer from Connection instead
  • Consider grouping primitive types in Session constructor into a type / namedtuple ConnectionParams
  • Evaluate setting session closed flag in the beginning of session close() (currently set at the end)
    • decision: the session closed flag should be set at the end of the session's close method. The flag should only be set if the session has successfully closed. Setting the flag at the beginning would lead to a resource leak because if the session closure fails and the user attempts to clean up resources by attempting to close() again, our code would disallow another request being made because we already consider the session to be closed.
      • pros: logical coherence - the session is only marked as closed if it successfully gets closed + maintenance of existing behaviour
      • cons: the session may not actually get closed despite the user attempting a close().
  • Remove service specific state from ExecuteResponse
  • Re-organise sea/ and backend/ relation

How is this tested?

  • Unit tests
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

PECOBLR-531
PECOBLR-447
PECOBLR-448
PECOBLR-603
PECOBLR-606

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>
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>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Copy link

Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase (git rebase -i main).

@varun-edachali-dbx varun-edachali-dbx deleted the sea-cleanup branch August 4, 2025 02:39
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