[PR] Avoid meaningless logging and patching

[kopf-archiver]

A pull request by nolar at 2019-11-15 10:44:05+00:00
Original URL: zalando-incubator/kopf#236
Merged by nolar at 2019-11-15 11:42:02+00:00

Less patching, less logging when not needed.

Issue : #1234 (only if appropriate)

Description

Skip logging the empty diffs. They are useless. Especially on the resuming events, when the object is not changed actually. — As they are already skipped for the creation/deletion events, when the diff is a full-body vs. None diff.

Skip patching the body essence if it is the same as the one already stored on the object. This pollutes the logs with difficult-to-read JSON dumps, especially on the resuming events — when these patches and logs contain no useful information.

Skip purging the handler progress if there is no progress stored (e.g. for one-handler cases, when it succeeds immediately). This produced unnecessary API patches, and polluted the logs for no good reason.

These changes will reduce the amount of meaningless logs produced.

Types of Changes

• Refactor/improvements

Review

List of tasks the reviewer must do to review the PR

• Tests