[GitHub] [airflow] jaegwonseo commented on pull request #33796: Add druid ingestion connection test

["jaegwonseo (via GitHub)" <gi...@apache.org>]

jaegwonseo commented on PR #33796:
URL: https://github.com/apache/airflow/pull/33796#issuecomment-1695791861

   > I think better approach woudl be to keep one of the connections (DBApi ?) as "druid". It will at least make existing DBApi connections work.
   
   > Also - this one woll need a changelog entry describing what changed and how users should modify their connections
   
   @potiuk 
   In order to execute queries within a distributed system, Druid utilizes broker nodes, and for data ingestion, it leverages ingest nodes. Since each node has distinct roles and invocation addresses, maintaining the current setup with these two separate nodes would be preferable.
   and I do not know the exact criteria for releasing new versions in the airflow provider, so I assumed 3.6.0 for now and added the change log.
   
   
   


-- 
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 unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org