Skip to content

Commit

Permalink
[hotfix] sqllab presto
Browse files Browse the repository at this point in the history
  • Loading branch information
mistercrunch committed Oct 21, 2016
1 parent 93f8e7d commit e9d4749
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion caravel/db_engine_specs.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def convert_dttm(cls, target_type, dttm):
return "'{}'".format(dttm.strftime('%Y-%m-%d %H:%M:%S'))

@classmethod
def handle_cursor(cls, cursor, query):
def handle_cursor(cls, cursor, query, session):
"""Handle a live cursor between the execute and fetchall calls
The flow works without this method doing anything, but it allows
Expand Down
2 changes: 1 addition & 1 deletion caravel/sql_lab.py
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ def handle_error(msg):
cursor = result_proxy.cursor
query.status = QueryStatus.RUNNING
session.flush()
db_engine_spec.handle_cursor(cursor, query)
db_engine_spec.handle_cursor(cursor, query, session)

cdf = None
if result_proxy.cursor:
Expand Down

0 comments on commit e9d4749

Please sign in to comment.