hussein-awala commented on code in PR #71794:
URL: https://github.com/apache/airflow/pull/71794#discussion_r3811251561


##########
providers/apache/kafka/docs/changelog.rst:
##########
@@ -27,6 +27,19 @@
 Changelog
 ---------
 
+1.16.0
+......
+
+Features
+~~~~~~~~
+
+* ``Kafka: Return consumed message results on request (#69740)``

Review Comment:
   Good catch — rewritten to name the class and the new parameter:
   
   `Add 'return_apply_function_results' to ConsumeFromTopicOperator to return 
apply_function results (#69740)`
   
   The PR adds an opt-in `return_apply_function_results: bool = False` to
   `ConsumeFromTopicOperator`; when enabled, non-`None` results from the 
per-message
   `apply_function` are collected and returned (and so land in XCom). The 
default preserves the
   previous `return None` behaviour. Also dropped the redundant `Kafka:` 
prefix, since this is
   the Kafka provider's own changelog.
   
   ---
   Drafted-by: Claude Code (Opus 5) (no human review before posting)
   



##########
providers/apache/livy/docs/changelog.rst:
##########
@@ -28,6 +28,24 @@
 Changelog
 ---------
 
+4.6.1
+.....
+
+Bug Fixes
+~~~~~~~~~
+
+* ``Make 'durable' reach 'default_args' and warn when set below Airflow 3.3 
(#71531)``
+
+Doc-only
+~~~~~~~~
+
+* ``Update changelog with better wording (#71161)``

Review Comment:
   Done, and applied across all providers as you suggested.
   
   `Update changelog with better wording (#71161)` is changelog maintenance 
rather than a
   user-facing doc change, so it now sits under the excluded marker — the same 
treatment the
   `Prepare providers release ...` commits get. Moved in all six providers 
where it was visible:
   `amazon`, `databricks`, `apache.livy`, `common.sql`, `microsoft.azure`, 
`microsoft.mssql`.
   
   For `apache.livy` that was the only `Doc-only` entry, so the heading is gone 
entirely;
   `amazon` and `microsoft.mssql` keep theirs since they have real doc entries.
   
   ---
   Drafted-by: Claude Code (Opus 5) (no human review before posting)
   



##########
providers/apache/kafka/docs/changelog.rst:
##########
@@ -27,6 +27,19 @@
 Changelog
 ---------
 
+1.16.0
+......
+
+Features
+~~~~~~~~
+
+* ``Kafka: Return consumed message results on request (#69740)``

Review Comment:
   Good catch — rewritten to name the class and the new parameter:
   
   `Add 'return_apply_function_results' to ConsumeFromTopicOperator to return 
apply_function results (#69740)`
   
   The PR adds an opt-in `return_apply_function_results: bool = False` to
   `ConsumeFromTopicOperator`; when enabled, non-`None` results from the 
per-message
   `apply_function` are collected and returned (and so land in XCom). The 
default preserves the
   previous `return None` behaviour. Also dropped the redundant `Kafka:` 
prefix, since this is
   the Kafka provider's own changelog.
   
   ---
   Drafted-by: Claude Code (Opus 5) (no human review before posting)
   



-- 
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