Open andy31415 opened 2 weeks ago
For the 3rd item, I could not find an immediate dedup logic for mLastSuccessfullyWrittenPath and mProcessingAttributePath as their code paths are updated at separate times. Deduplicating members within our handlers will likely require separate consideration.
Once we have a single codepath, some optimizations become clearer:
reporting/Read.cpp
is not ideally named, should be maybe inline into reporting or we should figure out proper naming and placement