[GitHub] [skywalking] Ax1an commented on a change in pull request #5699: Add syncBulkActions config to avoid the large amount of metrics data written ES in a single sync bulk request.

[GitBox <gi...@apache.org>]

Ax1an commented on a change in pull request #5699:
URL: https://github.com/apache/skywalking/pull/5699#discussion_r508564225



##########
File path: oap-server/server-storage-plugin/storage-elasticsearch-plugin/src/main/java/org/apache/skywalking/oap/server/storage/plugin/elasticsearch/StorageModuleElasticsearchConfig.java
##########
@@ -52,6 +52,7 @@
     private int superDatasetIndexShardsFactor = 5;
     private int indexRefreshInterval = 2;
     private int bulkActions = 2000;
+    private int syncBulkActions = 10000;

Review comment:
       In our production environment, it's 100k. However, considering that most people try skywalking, the machine configuration is not so high, and this is a configurable item, so the initial value is set to 10k.




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

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