René's URL Explorer Experiment


Title: [471] - Close underlying HTTP Client on closing `Connection` by Varun0157 · Pull Request #674 · databricks/databricks-sql-python · GitHub

Open Graph Title: [471] - Close underlying HTTP Client on closing `Connection` by Varun0157 · Pull Request #674 · databricks/databricks-sql-python

X Title: [471] - Close underlying HTTP Client on closing `Connection` by Varun0157 · Pull Request #674 · databricks/databricks-sql-python

Description: (Addresses #471). What type of PR is this? Bug Fix Description Here, we resolve a few resource leaks with regards to TCP connections remaining open even after the Connection is closed: Currentl...

Open Graph Description: (Addresses #471). What type of PR is this? Bug Fix Description Here, we resolve a few resource leaks with regards to TCP connections remaining open even after the Connection is closed: Currentl...

X Description: (Addresses #471). What type of PR is this? Bug Fix Description Here, we resolve a few resource leaks with regards to TCP connections remaining open even after the Connection is closed: Currentl...

Opengraph URL: https://github.com/databricks/databricks-sql-python/pull/674

X: @github

direct link

Domain: patch-diff.githubusercontent.com

route-pattern/_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format)
route-controllervoltron_pull_requests_fragments
route-actionpull_request_layout
fetch-noncev2:8ceadac6-3172-069c-3ae2-3c96f5869d6b
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-id9642:2A625F:2B6091B:3D13E5A:6971075B
html-safe-nonce93a141f034b2da21ff3995822d8f94d7ac7111b91216dc8018621c2c835ed5e4
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5NjQyOjJBNjI1RjoyQjYwOTFCOjNEMTNFNUE6Njk3MTA3NUIiLCJ2aXNpdG9yX2lkIjoiODE4NzEzOTA0NzUyNDI4ODkxIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0=
visitor-hmac07a16673f8952762ca62954b0f72cdbdbdbaa0d873d8cfa6334b643eaf0c11e2
hovercard-subject-tagpull_request:2732881075
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/pull_requests_fragments/pull_request_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/databricks/databricks-sql-python/674/pull_request_layout
twitter:imagehttps://avatars.githubusercontent.com/u/4998052?s=400&v=4
twitter:cardsummary
og:imagehttps://avatars.githubusercontent.com/u/4998052?s=400&v=4
og:image:alt(Addresses #471). What type of PR is this? Bug Fix Description Here, we resolve a few resource leaks with regards to TCP connections remaining open even after the Connection is closed: Currentl...
og:site_nameGitHub
og:typeobject
og:author:usernameVarun0157
hostnamegithub.com
expected-hostnamegithub.com
None6d0e7b0d5e294a53c55f49e80755e785182080c6e6a48c9ed353bedddaee69f4
turbo-cache-controlno-preview
go-importgithub.com/databricks/databricks-sql-python git https://github.com/databricks/databricks-sql-python.git
octolytics-dimension-user_id4998052
octolytics-dimension-user_logindatabricks
octolytics-dimension-repository_id493695132
octolytics-dimension-repository_nwodatabricks/databricks-sql-python
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id493695132
octolytics-dimension-repository_network_root_nwodatabricks/databricks-sql-python
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release6d5a3a396bc74e65fd3ef897b0a99a6e489debaa
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#start-of-content
https://patch-diff.githubusercontent.com/
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fdatabricks%2Fdatabricks-sql-python%2Fpull%2F674
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub SparkBuild and deploy intelligent appshttps://github.com/features/spark
GitHub ModelsManage and compare promptshttps://github.com/features/models
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fdatabricks%2Fdatabricks-sql-python%2Fpull%2F674
Sign up https://patch-diff.githubusercontent.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=databricks%2Fdatabricks-sql-python
Reloadhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Reloadhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Reloadhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
databricks https://patch-diff.githubusercontent.com/databricks
databricks-sql-pythonhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python
Notifications https://patch-diff.githubusercontent.com/login?return_to=%2Fdatabricks%2Fdatabricks-sql-python
Fork 132 https://patch-diff.githubusercontent.com/login?return_to=%2Fdatabricks%2Fdatabricks-sql-python
Star 218 https://patch-diff.githubusercontent.com/login?return_to=%2Fdatabricks%2Fdatabricks-sql-python
Code https://patch-diff.githubusercontent.com/databricks/databricks-sql-python
Issues 85 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/issues
Pull requests 44 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pulls
Actions https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/actions
Projects 0 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/projects
Wiki https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/wiki
Security Uh oh! There was an error while loading. Please reload this page. https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/security
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Insights https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pulse
Code https://patch-diff.githubusercontent.com/databricks/databricks-sql-python
Issues https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/issues
Pull requests https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pulls
Actions https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/actions
Projects https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/projects
Wiki https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/wiki
Security https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/security
Insights https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pulse
Sign up for GitHub https://patch-diff.githubusercontent.com/signup?return_to=%2Fdatabricks%2Fdatabricks-sql-python%2Fissues%2Fnew%2Fchoose
terms of servicehttps://docs.github.com/terms
privacy statementhttps://docs.github.com/privacy
Sign inhttps://patch-diff.githubusercontent.com/login?return_to=%2Fdatabricks%2Fdatabricks-sql-python%2Fissues%2Fnew%2Fchoose
Jump to bottomhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#issue-comment-box
Varun0157https://patch-diff.githubusercontent.com/Varun0157
databricks:mainhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/tree/main
Varun0157:close-connhttps://patch-diff.githubusercontent.com/Varun0157/databricks-sql-python/tree/close-conn
[471] - Close underlying HTTP Client on closing Connection https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#top
Varun0157https://patch-diff.githubusercontent.com/Varun0157
databricks:mainhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/tree/main
Varun0157:close-connhttps://patch-diff.githubusercontent.com/Varun0157/databricks-sql-python/tree/close-conn
Conversation 18 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Commits 43 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits
Checks 0 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/checks
Files changed https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
https://github.co/hiddenchars
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/{{ revealButtonHref }}
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
Aug 9, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#issue-3306682996
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
#471https://github.com/databricks/databricks-sql-python/issues/471
TCP connection not closing properly leading to dangling CLOSE_WAIT state and system overload #471https://github.com/databricks/databricks-sql-python/issues/471
#471https://github.com/databricks/databricks-sql-python/issues/471
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
vikrantpuppalahttps://patch-diff.githubusercontent.com/vikrantpuppala
August 8, 2025 12:58https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-4437a2a
https://patch-diff.githubusercontent.com/vikrantpuppala
Refactor codebase to use a unified http clienthttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4437a2ac0b46b700a65fb8ad946c97b8f212c52e
4437a2ahttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4437a2ac0b46b700a65fb8ad946c97b8f212c52e
https://patch-diff.githubusercontent.com/vikrantpuppala
Some more fixes and aligned testshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/30c04a66c7abd88f455b57d78dd2ae230ff4b0cc
30c04a6https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/30c04a66c7abd88f455b57d78dd2ae230ff4b0cc
https://patch-diff.githubusercontent.com/vikrantpuppala
Fix all testshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/429460082749de360c9e86e55772f093deeca05e
4294600https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/429460082749de360c9e86e55772f093deeca05e
https://patch-diff.githubusercontent.com/vikrantpuppala
fmthttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/31552117d01160d59980a201a5c47d7135eb4040
3155211https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/31552117d01160d59980a201a5c47d7135eb4040
https://patch-diff.githubusercontent.com/Varun0157
preliminary connection closure funchttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1143838ad277f4a0309fdb40cdf682ad8e98ad9a
1143838https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1143838ad277f4a0309fdb40cdf682ad8e98ad9a
https://patch-diff.githubusercontent.com/Varun0157
unit test for backend closurehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/68cc8221457f75ac22b1ac6d877d9982f8dae2e5
68cc822https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/68cc8221457f75ac22b1ac6d877d9982f8dae2e5
https://patch-diff.githubusercontent.com/Varun0157
remove redundant commenthttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ef1d9fd0fdfc2af5387786d24372726f69866091
ef1d9fdhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ef1d9fd0fdfc2af5387786d24372726f69866091
https://patch-diff.githubusercontent.com/Varun0157
assert SEA http client closure in unit testshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4bb2e4b0fb4238612d9b1f5b8401706a5295113c
4bb2e4bhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4bb2e4b0fb4238612d9b1f5b8401706a5295113c
https://patch-diff.githubusercontent.com/Varun0157
correct docstrnghttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/734dd06e131b274b6af7e5fab58dc4fabaa4902f
734dd06https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/734dd06e131b274b6af7e5fab58dc4fabaa4902f
https://patch-diff.githubusercontent.com/vikrantpuppala
fix e2ehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d00e3c86a798bc8c7f89c9cc59fcae20ef7eff61
d00e3c8https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d00e3c86a798bc8c7f89c9cc59fcae20ef7eff61
https://patch-diff.githubusercontent.com/vikrantpuppala
fix unithttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/000d3a360000c0b7f3c5914d061296685224cf5f
000d3a3https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/000d3a360000c0b7f3c5914d061296685224cf5f
https://patch-diff.githubusercontent.com/vikrantpuppala
more fixeshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/cba3da70ce26696b645debd3a6d3dd523f7074f6
cba3da7https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/cba3da70ce26696b645debd3a6d3dd523f7074f6
https://patch-diff.githubusercontent.com/vikrantpuppala
more fixeshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/2a1f719025224d9ec4ae9e53fbb1359a193948b7
2a1f719https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/2a1f719025224d9ec4ae9e53fbb1359a193948b7
https://patch-diff.githubusercontent.com/vikrantpuppala
review commentshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1dd40a10b0227e837e5adda96816f2d54c22d58d
1dd40a1https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1dd40a10b0227e837e5adda96816f2d54c22d58d
https://patch-diff.githubusercontent.com/vikrantpuppala
fix warningshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/3847acac62be645cba8294c513399f75eb9bc1b2
3847acahttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/3847acac62be645cba8294c513399f75eb9bc1b2
https://patch-diff.githubusercontent.com/vikrantpuppala
fix check-typeshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d9a4797bd54632375eb3487233926c208e2abfbc
d9a4797https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d9a4797bd54632375eb3487233926c208e2abfbc
https://patch-diff.githubusercontent.com/vikrantpuppala
remove separate http client for telemetryhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ba2a3a9827478dad7e983c8d0ac4c9e49d4bcb21
ba2a3a9https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ba2a3a9827478dad7e983c8d0ac4c9e49d4bcb21
https://patch-diff.githubusercontent.com/vikrantpuppala
more clean uphttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d1f045ebe252883997bd1f67643da6da0cff5ada
d1f045ehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d1f045ebe252883997bd1f67643da6da0cff5ada
https://patch-diff.githubusercontent.com/Varun0157
Merge remote-tracking branch 'target/http-client-refactor-2' into clo…https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ea3b0b018147715e04dc495240f80e3a08e91f56
ea3b0b0https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ea3b0b018147715e04dc495240f80e3a08e91f56
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 07:31https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19130219353
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 07:31https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19130222775
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 13:03https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-4e66230
https://patch-diff.githubusercontent.com/Varun0157
remove excess release_connection callhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4e6623009be8c21c46088c4da420b1078dc80c59
4e66230https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4e6623009be8c21c46088c4da420b1078dc80c59
https://patch-diff.githubusercontent.com/Varun0157
Merge remote-tracking branch 'target/main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/bf0a2f605f00e78ac75b32556c04d1c11160418e
bf0a2f6https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/bf0a2f605f00e78ac75b32556c04d1c11160418e
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 15:43https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19140394421
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 15:43https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19140403701
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 13, 2025 21:59https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-67020f1
https://patch-diff.githubusercontent.com/Varun0157
formatting (black) - fix some closureshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/67020f1afb82dd15527a94ce90c97e3b5c6b6e2c
67020f1https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/67020f1afb82dd15527a94ce90c97e3b5c6b6e2c
https://patch-diff.githubusercontent.com/Varun0157
Revert "formatting (black) - fix some closures"https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/496d7f7e949b1e4a51c2d4eb2c2318752093eea6
496d7f7https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/496d7f7e949b1e4a51c2d4eb2c2318752093eea6
67020f1https://github.com/Varun0157/databricks-sql-python/commit/67020f1afb82dd15527a94ce90c97e3b5c6b6e2c
https://patch-diff.githubusercontent.com/Varun0157
add more http_client closureshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/84ec33a01ca33d0580e65aa686fe51921d094dd0
84ec33ahttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/84ec33a01ca33d0580e65aa686fe51921d094dd0
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
Aug 13, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#issuecomment-3184647755
@jprakash-dbhttps://github.com/jprakash-db
@jayantsing-dbhttps://github.com/jayantsing-db
@vikrantpuppalahttps://github.com/vikrantpuppala
#471https://github.com/databricks/databricks-sql-python/issues/471
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
https://patch-diff.githubusercontent.com/Varun0157
remove excess close callhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/76ce5ce3fe083b25297e142a22b65d759fab1556
76ce5cehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/76ce5ce3fe083b25297e142a22b65d759fab1556
https://patch-diff.githubusercontent.com/vikrantpuppala
vikrantpuppalahttps://patch-diff.githubusercontent.com/vikrantpuppala
Aug 14, 2025 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3118588435
View reviewed changes https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/76ce5ce3fe083b25297e142a22b65d759fab1556
vikrantpuppalahttps://patch-diff.githubusercontent.com/vikrantpuppala
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3118588435
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/76ce5ce3fe083b25297e142a22b65d759fab1556#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Varun0157https://patch-diff.githubusercontent.com/Varun0157
August 14, 2025 10:42https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-4452725
https://patch-diff.githubusercontent.com/Varun0157
wait for _flush before closing HTTP clienthttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4452725590dbd56d2b17dde1bd1c1e3f15c1ba3a
4452725https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/4452725590dbd56d2b17dde1bd1c1e3f15c1ba3a
https://patch-diff.githubusercontent.com/Varun0157
make close() asynchttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d90ac80a693e0c52ddb40551093225c4d0af5c60
d90ac80https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/d90ac80a693e0c52ddb40551093225c4d0af5c60
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
vikrantpuppalahttps://patch-diff.githubusercontent.com/vikrantpuppala
August 14, 2025 05:27https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19151451432
https://patch-diff.githubusercontent.com/jayantsing-db
jayantsing-dbhttps://patch-diff.githubusercontent.com/jayantsing-db
Aug 14, 2025 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3119367137
View reviewed changes https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/d90ac80a693e0c52ddb40551093225c4d0af5c60
src/databricks/sql/backend/sea/backend.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/d90ac80a693e0c52ddb40551093225c4d0af5c60#diff-5f9c9eec6231c8a82c4778821bca03bd96f1b1837882c00a9006df71287d254d
jayantsing-dbhttps://patch-diff.githubusercontent.com/jayantsing-db
Aug 14, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#discussion_r2275742302
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Varun0157https://patch-diff.githubusercontent.com/Varun0157
Aug 14, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#discussion_r2275837637
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
jayantsing-dbhttps://patch-diff.githubusercontent.com/jayantsing-db
August 15, 2025 13:29https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-19176146879
Varun0157https://patch-diff.githubusercontent.com/Varun0157
September 3, 2025 10:47https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-8ff6552
https://patch-diff.githubusercontent.com/Varun0157
Merge remote-tracking branch 'target/main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/8ff6552b992daef142d42edb690648a1eb685d54
8ff6552https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/8ff6552b992daef142d42edb690648a1eb685d54
https://patch-diff.githubusercontent.com/Varun0157
Merge remote-tracking branch 'origin/main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/666fe6254ea082c2b5db8d0a91cb97fd34c9aae8
666fe62https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/666fe6254ea082c2b5db8d0a91cb97fd34c9aae8
https://patch-diff.githubusercontent.com/Varun0157
Merge branch 'main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1c88a01f7a0a4b43a23df1961e1b929ade376aca
1c88a01https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/1c88a01f7a0a4b43a23df1961e1b929ade376aca
https://patch-diff.githubusercontent.com/Varun0157
Merge branch 'main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/f78456b242f1666f31a45d7cea432c6f453c88e3
f78456bhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/f78456b242f1666f31a45d7cea432c6f453c88e3
https://patch-diff.githubusercontent.com/Varun0157
Merge branch 'main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/b97ac038e76ef3d89564d4f6d2e19b6807d37cda
b97ac03https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/b97ac038e76ef3d89564d4f6d2e19b6807d37cda
https://patch-diff.githubusercontent.com/Varun0157
simplify close_session (remove secondary _close_session invocation)https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/c78bace62b05f6d47cae47dbbc2cb4c9a31ef5de
c78bacehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/c78bace62b05f6d47cae47dbbc2cb4c9a31ef5de
https://patch-diff.githubusercontent.com/Varun0157
simplify changeshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/bedfc06aa2e14cdc3e675f1546795b6d582b53c6
bedfc06https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/bedfc06aa2e14cdc3e675f1546795b6d582b53c6
https://patch-diff.githubusercontent.com/Varun0157
simplify diffhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/a36353b7938867311798f49bc628743dd373623b
a36353bhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/a36353b7938867311798f49bc628743dd373623b
https://patch-diff.githubusercontent.com/Varun0157
simplify imports, log TelemetryClient closurehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/76fb62340d491281374c34f30b97432d603fd00c
76fb623https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/76fb62340d491281374c34f30b97432d603fd00c
https://patch-diff.githubusercontent.com/Varun0157
simplify diffhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/66192b4bda01f0684cd5c6a458a7a9b9b3103a99
66192b4https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/66192b4bda01f0684cd5c6a458a7a9b9b3103a99
https://patch-diff.githubusercontent.com/Varun0157
Varun0157https://patch-diff.githubusercontent.com/Varun0157
Oct 28, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#issuecomment-3456276515
@jayantsing-dbhttps://github.com/jayantsing-db
@vikrantpuppalahttps://github.com/vikrantpuppala
@jprakash-dbhttps://github.com/jprakash-db
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
https://patch-diff.githubusercontent.com/Varun0157
Merge branch 'main' into close-connhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/171439523fc346b7a9243644c2e77fabbc16fb76
1714395https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/171439523fc346b7a9243644c2e77fabbc16fb76
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
November 30, 2025 19:41https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-21259484172
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Nov 30, 2025 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522177469
View reviewed changes https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/171439523fc346b7a9243644c2e77fabbc16fb76
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522177469
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
#471https://github.com/databricks/databricks-sql-python/issues/471
Add Copilot custom instructionshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/new/main/.github/instructions?filename=*.instructions.md
Learn how to get startedhttps://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/171439523fc346b7a9243644c2e77fabbc16fb76#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/backend/sea/backend.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/171439523fc346b7a9243644c2e77fabbc16fb76#diff-5f9c9eec6231c8a82c4778821bca03bd96f1b1837882c00a9006df71287d254d
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/171439523fc346b7a9243644c2e77fabbc16fb76#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/171439523fc346b7a9243644c2e77fabbc16fb76#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Varun0157https://patch-diff.githubusercontent.com/Varun0157
December 1, 2025 01:14https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-7b4f6cb
https://patch-diff.githubusercontent.com/Varun0157
Revert "Merge branch 'main' into close-conn"https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/7b4f6cbe6d51acf793e51d2f1ae629ca0db699d9
7b4f6cbhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/7b4f6cbe6d51acf793e51d2f1ae629ca0db699d9
1714395https://github.com/Varun0157/databricks-sql-python/commit/171439523fc346b7a9243644c2e77fabbc16fb76
66192b4https://github.com/Varun0157/databricks-sql-python/commit/66192b4bda01f0684cd5c6a458a7a9b9b3103a99
https://patch-diff.githubusercontent.com/Varun0157
Reapply "Merge branch 'main' into close-conn"https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/6187d06a462b6e7412be64c0589c56573ec6d0ac
6187d06https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/6187d06a462b6e7412be64c0589c56573ec6d0ac
7b4f6cbhttps://github.com/Varun0157/databricks-sql-python/commit/7b4f6cbe6d51acf793e51d2f1ae629ca0db699d9
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
November 30, 2025 19:46https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-21259530658
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Nov 30, 2025 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522188304
View reviewed changes https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/6187d06a462b6e7412be64c0589c56573ec6d0ac
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522188304
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Add Copilot custom instructionshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/new/main/.github/instructions?filename=*.instructions.md
Learn how to get startedhttps://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/6187d06a462b6e7412be64c0589c56573ec6d0ac#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Varun0157https://patch-diff.githubusercontent.com/Varun0157
December 1, 2025 01:25https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#commits-pushed-f7af379
https://patch-diff.githubusercontent.com/Varun0157
simplify diffhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/f7af379316c4c1d86fc76cc59a92f512a52c0df6
f7af379https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/f7af379316c4c1d86fc76cc59a92f512a52c0df6
https://patch-diff.githubusercontent.com/Varun0157
black srchttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/de2f848a2a803a40879e722d4f76b2f2d682b800
de2f848https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/de2f848a2a803a40879e722d4f76b2f2d682b800
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
November 30, 2025 19:56https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#event-21259606413
https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Nov 30, 2025 https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522209922
View reviewed changes https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/de2f848a2a803a40879e722d4f76b2f2d682b800
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#pullrequestreview-3522209922
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Add Copilot custom instructionshttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/new/main/.github/instructions?filename=*.instructions.md
Learn how to get startedhttps://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/telemetry/telemetry_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/de2f848a2a803a40879e722d4f76b2f2d682b800#diff-2e5fda1fd47b91b20859af61882619047817b04d78a60957f85f6dd24f4f34d5
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
src/databricks/sql/backend/sea/utils/http_client.pyhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/de2f848a2a803a40879e722d4f76b2f2d682b800#diff-fb0f0ae9af55442d437f95075c2a3c2412c1d7fcf6dfb0fd79d485b5192b347b
Copilothttps://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Nov 30, 2025https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674#discussion_r2574692595
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
Copilothttps://docs.github.com/copilot/responsible-use-of-github-copilot-features/responsible-use-of-github-copilot-code-review
https://patch-diff.githubusercontent.com/Varun0157
remove excess ThreadPoolExcecutor importhttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ec991fe20d3764df60863c9c41fe6488e34a1906
ec991fehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/commits/ec991fe20d3764df60863c9c41fe6488e34a1906
Sign up for freehttps://patch-diff.githubusercontent.com/join?source=comment-repo
Sign in to commenthttps://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fdatabricks%2Fdatabricks-sql-python%2Fpull%2F674
Copilot code review https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
Copilot https://patch-diff.githubusercontent.com/apps/copilot-pull-request-reviewer
https://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674/files/de2f848a2a803a40879e722d4f76b2f2d682b800
https://patch-diff.githubusercontent.com/vikrantpuppala
vikrantpuppala https://patch-diff.githubusercontent.com/vikrantpuppala
https://patch-diff.githubusercontent.com/jayantsing-db
jayantsing-db https://patch-diff.githubusercontent.com/jayantsing-db
Please reload this pagehttps://patch-diff.githubusercontent.com/databricks/databricks-sql-python/pull/674
https://patch-diff.githubusercontent.com/Varun0157
https://patch-diff.githubusercontent.com/vikrantpuppala
https://patch-diff.githubusercontent.com/jayantsing-db
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width

URLs of crawlers that visited me.