Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

cleanup(batcher): removing the non-applicable TODOs comments #834

Merged
merged 2 commits into from Dec 27, 2019
Merged

cleanup(batcher): removing the non-applicable TODOs comments #834

merged 2 commits into from Dec 27, 2019

Commits on Dec 16, 2019

  1. fix(batcher): fix logged messge when exception occurs post processing 

    This commit addresses logged message while batch post
processing.

Currently, if a batch is successful and some exception
occurs during processing then the Batcher generates an exception message
with "Batching finished with 3 batches failed" which is not an accurate
message.

After this commit, for the same scenario, the message will be
logged with "Batching finished with 3 partial failures...".
    @rahulKQL
    rahulKQL committed Dec 16, 2019
    Copy the full SHA
    1dfefca View commit details
    Browse the repository at this point in the history

Commits on Dec 18, 2019

  1. removed unnecessary batcher message changes, Also reverted the Batche… 

    …rStats method name to `recordBatchElementsCompletion()`.
    @rahulKQL
    rahulKQL committed Dec 18, 2019
    Copy the full SHA
    8f34639 View commit details
    Browse the repository at this point in the history