R4R: Remove KV marshalling split

[rigelrozanski]

closes #3233

• Targeted PR against correct branch (see CONTRIBUTING.md)

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

• Wrote tests

• Updated relevant documentation (docs/)

• Added entries in PENDING.md with issue #

• rereviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[jackzampolin]

@rigelrozanski can you link to an issue that describes this work, or summarize it in the PR description please?

[alexanderbez]

+79 −323 ❤️

[rigelrozanski]

@jackzampolin yeah I'm going to dig up the issue or create one if it doesn't exist - this issue had been fleshed out in several sdk team discussions to date - also is prerequisite to making #1402 a simpler implementation.

[jackzampolin]

Thank you @rigelrozanski !

[codecov]

Codecov Report

Merging #3230 into develop will decrease coverage by 0.01%.
The diff coverage is 81.81%.

@@             Coverage Diff             @@
##           develop    #3230      +/-   ##
===========================================
- Coverage    54.88%   54.86%   -0.02%     
===========================================
  Files          133      133              
  Lines         9555     9552       -3     
===========================================
- Hits          5244     5241       -3     
  Misses        3989     3989              
  Partials       322      322

[jackzampolin]

Good looking refactor and cleanup. Working for me and removes a bunch of code!