Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428246056
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-def
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428247519
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-def
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428885237
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-def
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428247519
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-def
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428246056
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-def
bolkedebruin commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428218892
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-d
bolkedebruin commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428216767
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-d
bolkedebruin commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1428216767
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -241,13 +258,17 @@ def run(
else:
return results
-@staticmethod
-d
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1427221843
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -80,8 +85,18 @@ def __init__(
self.http_headers = http_headers
self.catalog = cata
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1427221843
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -80,8 +85,18 @@ def __init__(
self.http_headers = http_headers
self.catalog = cata
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1427222586
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -80,8 +85,18 @@ def __init__(
self.http_headers = http_headers
self.catalog = cata
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1427221843
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -80,8 +85,18 @@ def __init__(
self.http_headers = http_headers
self.catalog = cata
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1856184404
Constraints with databricks 5.0.0 updated
https://github.com/apache/airflow/commit/33121932d4cdcd2da27756ac149791365a88f08c
and tag for 2.8.0rc3 moved @ephraimbuddy
--
This is an aut
bolkedebruin commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1426953507
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -80,8 +85,18 @@ def __init__(
self.http_headers = http_headers
self.catalog = ca
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1856011705
Also cc: @ephraimbuddy -> since we yanked 5.0.1, I will rerun the last
build for v2-8-stable to refresh constraints and move the tag once done
(currentlly 5.0.1 was in the constraints p
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1856003948
Yanked
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscri
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1855996509
> The databricks provider was already released with the breaking change->
[5.0.1](https://pypi.org/project/apache-airflow-providers-databricks/5.0.1/).
Maybe this version should be remov
Joffreybvn commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1855988794
The databricks provider was already released with the breaking change->
[5.0.1](https://pypi.org/project/apache-airflow-providers-databricks/5.0.1/).
Maybe this version should be rem
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1855968176
And I created https://github.com/apache/airflow/issues/36224 and assigned to
myself and will shortly take a close look at this - to make sure the common.sql
API is not only defined but a
potiuk commented on PR #36205:
URL: https://github.com/apache/airflow/pull/36205#issuecomment-1855953848
@bolkedebruin @phanikumv -> please take a look and see what you think. I am
all for this direction
--
This is an automated message from the Apache Git Service.
To respond to the messag
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1425894982
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -243,11 +244,15 @@ def run(
@staticmethod
def _make_serializable(result):
-"""Tr
Joffreybvn commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1425862433
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -243,11 +244,15 @@ def run(
@staticmethod
def _make_serializable(result):
-"""Tr
potiuk commented on code in PR #36205:
URL: https://github.com/apache/airflow/pull/36205#discussion_r1425659482
##
airflow/providers/databricks/hooks/databricks_sql.py:
##
@@ -243,11 +244,15 @@ def run(
@staticmethod
def _make_serializable(result):
-"""Transf
23 matches
Mail list logo