Skip to content

Commit

Permalink
dialect can be regular method
Browse files Browse the repository at this point in the history
  • Loading branch information
jonmmease committed Aug 10, 2023
1 parent 684df5c commit 513c3cc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions python/vegafusion/vegafusion/dataset/duckdb.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@


class DuckDbDataset(SqlDataset):
@classmethod
def dialect(cls) -> str:
def dialect(self) -> str:
return "duckdb"

def __init__(self, relation: DuckDBPyRelation, fallback: bool = True, verbose: bool = False):
Expand Down
3 changes: 1 addition & 2 deletions python/vegafusion/vegafusion/dataset/sql.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ class SqlDataset(ABC):
Python interface for VegaFusion Sql Dataset
"""

@classmethod
def dialect(cls) -> str:
def dialect(self) -> str:
"""
Returns SQL dialect accepted by the connection
Expand Down

0 comments on commit 513c3cc

Please sign in to comment.