Closed Phoenix500526 closed 2 months ago
Attention: Patch coverage is 57.98319%
with 50 lines
in your changes are missing coverage. Please review.
Project coverage is 75.59%. Comparing base (
e35b35a
) to head (8f4f038
). Report is 14 commits behind head on master.:exclamation: Current head 8f4f038 differs from pull request most recent head ebda1ef. Consider uploading reports for the commit ebda1ef to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Refs: #774 Closes: #774
Please briefly answer these questions:
what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?) FYI: #774
what changes does this pull request make? Add a retry logic in
write_batch
.are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc) There is nothing to change breakingly, but we need to do a performance regression test for this modification.