-
Notifications
You must be signed in to change notification settings - Fork 205
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Don't duplicate log failed futures in the http json api (commandservi…
…ce specific ones) (#9990) * Don't duplicate log failed futures in the http json api (commandservice specific) changelog_begin - [JSON API] Errors which arise from the CommandService are not logged twice anymore (thus reducing noise) changelog_end * Fix duplicate logging of a failed future differently via adding another error case in CommandService.scala * Restore old formatting * Improve comment * Remove the wrong changes I accidentally made, the functions are quite similar xD * remove the additional type and just make the id optional & remove unecessary comment
- Loading branch information
1 parent
0c32cf2
commit 3d2d4b9
Showing
2 changed files
with
36 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters