jaegwonseo commented on code in PR #33796:
URL: https://github.com/apache/airflow/pull/33796#discussion_r1314253423


##########
airflow/providers/apache/druid/CHANGELOG.rst:
##########
@@ -27,6 +27,19 @@
 Changelog
 ---------
 
+4.0.0
+.....
+
+Features
+~~~~~~~~
+
+* ``Add druid ingestion connection test And expose druid ingest hook (#33795)``
+
+.. note::
+  The connection type of Druid has been separated into two(druid_broker, 
druid_ingest)

Review Comment:
   @eladkal @Taragolis 
   Taragolis thanks for adding example link
   in this pr the only concern point is changing connection type. So, I think 
this mention is enough.
   ```
   In this version of provider Apache Druid Connection(conn_type="druid") has 
been separated into two(Druid Broker, Druid Ingest)
   Update connect type(conn_type = "druid") to druid broker, druid ingest 
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to