Conversation
eb92ca6 to
3fbc3fe
Compare
🔍 Preview links for changed docs |
|
Updated to match pattern for rolling up previous minor patch releases into new minor release notes. #17823 |
docs/release-notes/index.md
Outdated
|
|
||
| * Introduces `proxy` param to support proxy [#316](https://github.com/elastic/logstash-filter-elastic_integration/pull/316) | ||
| * Embeds Ingest Node components from Elasticsearch 9.1 | ||
| * Introduces `proxy` config to support proxy URI to connect to Elasticsearch. [#320](https://github.com/elastic/logstash-filter-elastic_integration/pull/320) |
docs/release-notes/index.md
Outdated
| * Significantly improves write speeds to the persistent queue (PQ) when a pipeline's workers are caught up with already-written events [#17791](https://github.com/elastic/logstash/pull/17791) | ||
| * Eliminated log warning about unknown gauge metric type when using pipeline-to-pipeline. [#17721](https://github.com/elastic/logstash/pull/17721) | ||
| * Improve plugins remove command to support multiple plugins [#17030](https://github.com/elastic/logstash/pull/17030) | ||
| * Dropped the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759) |
There was a problem hiding this comment.
| * Dropped the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759) | |
| * Deprecated the persistent queue setting `queue.checkpoint.interval`[#17759](https://github.com/elastic/logstash/pull/17759), which was found to have no effect. This will be removed in a future Logstash release. |
Let's soften the language here a little, as this is only a deprecation at the moment and note that the setting didn't actually do anything...
Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
karenzone
left a comment
There was a problem hiding this comment.
Left some comments inline for consideration. Otherwise, LGTM.
General comment: We should align on tense for entries. In the first section alone, we have "improveS", "eliminateD", and "improve_" This is NOT blocking.
docs/release-notes/index.md
Outdated
|
|
||
| ##### ES|QL Support in Elasticsearch Input (Technical Preview) | ||
|
|
||
| * Support for using ES|QL queries in the [Elasticsearch input](https://www.elastic.co/guide/en/logstash/current/plugins-inputs-elasticsearch.html) to add improved flexibility when ingesting data from Elasticsearch is now in Technical Preview. |
There was a problem hiding this comment.
Not sure that linking to the highest level of the Elasticsearch input adds value is such a specific piece of content. Suggest either deep linking to new docs, or removing the link.
And/or adding a link to the PR where the work was done.
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
robbavey
left a comment
There was a problem hiding this comment.
Added a suggestion to follow up on @karenzone's suggestion to not link to ES Input front page for ES|QL support
Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
|
@Mergifyio backport main |
✅ Backports have been createdDetails
|
* Update release notes for 9.1.0 * Human based release note edits * Roll up 9.0.1 -> 9.0.4 RN into 9.1.0 RN * code review feedback * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> * Update docs/release-notes/index.md Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com> * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> * Update docs/release-notes/index.md Co-authored-by: Rob Bavey <rob.bavey@elastic.co> --------- Co-authored-by: logstashmachine <43502315+logstashmachine@users.noreply.github.com> Co-authored-by: donoghuc <cas.donoghue@gmail.com> Co-authored-by: Rob Bavey <rob.bavey@elastic.co> Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com> (cherry picked from commit f63da8a)
* Update release notes for 9.1.0 * Human based release note edits * Roll up 9.0.1 -> 9.0.4 RN into 9.1.0 RN * code review feedback * Update docs/release-notes/index.md * Update docs/release-notes/index.md * Update docs/release-notes/index.md * Update docs/release-notes/index.md --------- (cherry picked from commit f63da8a) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: logstashmachine <43502315+logstashmachine@users.noreply.github.com> Co-authored-by: donoghuc <cas.donoghue@gmail.com> Co-authored-by: Rob Bavey <rob.bavey@elastic.co> Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
No description provided.