Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
Accelerator
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
databricks
/
databricks-sql-python
Public
Notifications
You must be signed in to change notification settings
Fork
145
Star
233
Code
Issues
98
Pull requests
75
Actions
Projects
Wiki
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Wiki
Security and quality
Insights
SQLAlchemy 2: reorganise dialect files into a single directory
- #231
#231
Merged
susodapop
merged 13 commits into
main
databricks/databricks-sql-python:main
from
peco-1106
databricks/databricks-sql-python:peco-1106
Copy head branch name to clipboard
Sep 26, 2023
Conversation
Commits
13
(13)
Checks
Files changed
Merged
SQLAlchemy 2: reorganise dialect files into a single directory
#231
susodapop
merged 13 commits into
main
databricks/databricks-sql-python:main
from
peco-1106
databricks/databricks-sql-python:peco-1106
Copy head branch name to clipboard
Commits
Commits on Sep 26, 2023
Here's the layout recommended by SQLAlchemy's documentation:
Show description for 95ce138
Jesse Whitehouse
committed
95ce138
View commit details
Copy full SHA for 95ce138
Browse repository at this point
(1/x) Move dialect files one directory up so that dialect layout will
Show description for fd0edce
Jesse Whitehouse
committed
fd0edce
View commit details
Copy full SHA for fd0edce
Browse repository at this point
WIP: update default paramstyle to be compatible with DBR (ROLL ME BACK PLEASE)
Show description for c6fea52
Jesse Whitehouse
committed
c6fea52
View commit details
Copy full SHA for c6fea52
Browse repository at this point
Set DBAPI paramstyle to match what DBR supports
Show description for 08f551a
Jesse Whitehouse
committed
08f551a
View commit details
Copy full SHA for 08f551a
Browse repository at this point
Update sqlalchemy e2e tests to get valid signal after parameterized queries were implemented in DBR
Show description for 158abc6
Jesse Whitehouse
committed
158abc6
View commit details
Copy full SHA for 158abc6
Browse repository at this point
After PECO-1109, remove skip markers for tests that depend on DECIMAL types
Show description for 2232b15
Jesse Whitehouse
committed
2232b15
View commit details
Copy full SHA for 2232b15
Browse repository at this point
Move tests files all into the dialect directory
Show description for 73585fb
Jesse Whitehouse
committed
73585fb
View commit details
Copy full SHA for 73585fb
Browse repository at this point
No need to instruct pytest discovery to skip these tests now that they
Show description for a2dd5a7
Jesse Whitehouse
committed
a2dd5a7
View commit details
Copy full SHA for a2dd5a7
Browse repository at this point
Update the conf file and the instructions for running tests
Show description for ba4bb87
Jesse Whitehouse
committed
ba4bb87
View commit details
Copy full SHA for ba4bb87
Browse repository at this point
We see the following errors in the sqlalchemy test suite after merging server-side parameters
Show description for 994560b
Jesse Whitehouse
committed
994560b
View commit details
Copy full SHA for 994560b
Browse repository at this point
Instruct mypy to skip tests written by SQLAlchemy maintainers
Show description for 43c17c1
Jesse Whitehouse
committed
43c17c1
View commit details
Copy full SHA for 43c17c1
Browse repository at this point
Update mypy dependency to fix github actions error
Show description for 7f62f1b
Jesse Whitehouse
committed
7f62f1b
View commit details
Copy full SHA for 7f62f1b
Browse repository at this point
Rename file to match SQLAlchemy guideliens
Show description for a485bd2
Jesse Whitehouse
committed
a485bd2
View commit details
Copy full SHA for a485bd2
Browse repository at this point
You can’t perform that action at this time.