potiuk commented on issue #62794:
URL: https://github.com/apache/airflow/issues/62794#issuecomment-4010560786

   > Can we merge the release notes for 
[#61492](https://github.com/apache/airflow/pull/61492) and 
[#62562](https://github.com/apache/airflow/pull/62562) please? The first change 
was held back from the rc2 wave and is now fixed in rc3. There should only be 
one change for adding the new config setting IMO.
   
   @stephen-bracken 
   
   I am not sure what exactly you mean by "merge release notes" - but probably 
this: 
https://airflow.apache.org/docs/apache-airflow-providers-elasticsearch/stable/changelog.html
 - generally documentation. - if needed - should ba added in PR if we want it 
to appear in release notes, there is no "extra step" here about merging release 
notes. Merging PR is what constitutes release notes documentation.
   
   We had some extra description added - in another PR (on @eladkal request) 
that explained that finally ES issues should be fixed:
   
   > The write_to_es Elasticsearch task logging feature, was incompatible with 
Airflow 3. This is now fixed. Users must upgrade 
apache-airflow-providers-elasticsearch>=6.5.0 to pick up the new 
ElasticsearchRemoteLogIO class required by the fixed logging path. No 
configuration changes are needed — existing settings such as write_to_es, 
target_index, json_format, host_field, and offset_field continue to work as 
before. This fix will work with any airflow-airflow>=3.0 version.
   
   Both PRs you mentioned are listed in the changelog as well. 
   
   If that's what you meant - all that is released and docs are published.
   
   But maybe you thought about somethign else? 
   


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