Re: [PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-30 Thread via GitHub
dirrao closed pull request #37025: Update DatabricksSqlOperator to work with namedtuple URL: https://github.com/apache/airflow/pull/37025 -- 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

Re: [PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-30 Thread via GitHub
dirrao commented on PR #37025: URL: https://github.com/apache/airflow/pull/37025#issuecomment-1916724800 Closing and reopening PR to run the test cases again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-29 Thread via GitHub
SuperBo commented on PR #37025: URL: https://github.com/apache/airflow/pull/37025#issuecomment-1915809441 hi @Joffreybvn, I fixed the code style -- 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 t

Re: [PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-29 Thread via GitHub
Joffreybvn commented on PR #37025: URL: https://github.com/apache/airflow/pull/37025#issuecomment-1914962675 Just need to fix the code formatting. A run of `breeze static-checks` will do it :) -- This is an automated message from the Apache Git Service. To respond to the message, please l

[PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-26 Thread via GitHub
SuperBo opened a new pull request, #37025: URL: https://github.com/apache/airflow/pull/37025 Now, DatabricksSQLHook will return nametuples as results. So we change "row.asDict()" to "row._asdict() to accommodate these changes. We also add test cases for json and jsonl output, whi

Re: [PR] Update DatabricksSqlOperator to work with namedtuple [airflow]

2024-01-26 Thread via GitHub
boring-cyborg[bot] commented on PR #37025: URL: https://github.com/apache/airflow/pull/37025#issuecomment-1911663803 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution