sodadata / soda-sql

Soda SQL and Soda Spark have been deprecated and replaced by Soda Core. docs.soda.io/soda-core/overview.html
https://docs.soda.io/
Apache License 2.0
61 stars 17 forks source link

Use abstract Dialect class and methods instead of RuntimeError #158

Closed m1n0 closed 3 years ago

m1n0 commented 3 years ago

Dialects are controlling the mandatory abstract methods using RuntimeError - it would be neater to use abstract classes and methods, and it would prevent IDEs complaining about unreachable code after dialect.create_connection().