You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2022/04/20 03:22:11 UTC

[GitHub] [skywalking] wu-sheng commented on a diff in pull request #8913: Fix bulk processor initialize multiple times

wu-sheng commented on code in PR #8913:
URL: https://github.com/apache/skywalking/pull/8913#discussion_r853693784


##########
docs/en/changes/changes.md:
##########
@@ -21,7 +21,7 @@
 * Webapp module (for UI) enabled compression.
 * [Breaking Change] Add layer field to event, report an event without layer is not allowed.
 * Fix ES flush thread stops when flush schedule task throws exception, such as ElasticSearch flush failed.
-
+* Fix ES BulkProcessor in BatchProcessEsDAO Initialize multiple times and create multiple ES flush schedule task.

Review Comment:
   ```suggestion
   * Fix ES BulkProcessor in BatchProcessEsDAO was initialized multiple times and created multiple ES flush schedule tasks.
   
   ```



-- 
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: notifications-unsubscribe@skywalking.apache.org

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