New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

@W-13729777 - merge small indexes together #600

Merged

sagikSF merged 16 commits into master from W-13729777_index_merger

Aug 15, 2023

Collaborator

sagikSF commented Aug 9, 2023

No description provided.


          merge small indexes together

0a96cd7

salesforce-cla bot added the cla:signed label

sagikSF added 2 commits

August 9, 2023 13:14


          fix URI

09f5414


          change size to double

ca3fece

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved


          Update timbermill-java/timbermill-local/src/main/java/com/datorama/os…

5dd6f2f

…s/timbermill/ElasticsearchClient.java

Co-authored-by: JaneVelger <[email protected]>

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ermill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/cron/CronsRunner.java Show resolved Hide resolved

janevelger reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/cron/IndexMergerJob.java Show resolved Hide resolved

sagikSF added 10 commits

August 9, 2023 15:21


          add logs and code review

36de9c1


          Merge remote-tracking branch 'origin/W-13729777_index_merger' into W-…

fc19a9f

…13729777_index_merger

# Conflicts:
#	timbermill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java


          run reindex async

cd201e4


          set slices for reindex

9506c72


          add logs

ee184ee


          move delete to main thread

93be1a0


          extend ES clint socket timeout

f317af5


          extend ES clint socket timeout

a0da8c2


          send reindex as a task

e8061b0


          add sleep

975d8ff

tvaingart reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved

tvaingart reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Show resolved Hide resolved

tvaingart reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Show resolved Hide resolved

tvaingart approved these changes

View reviewed changes

tvaingart left a comment

@sagikSF approved but wrote some comment, if you wanna change


          add indexMergePercentage param

edengal reviewed

View reviewed changes

...ill-java/timbermill-local/src/main/java/com/datorama/oss/timbermill/ElasticsearchClient.java Outdated Show resolved Hide resolved

fix

843e7d2

sagikSF merged commit 8a2e102 into master

2 of 3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels