Christopher L. Shannon
Christopher L. Shannon
@ctubbsii and @dlmarion - here is my draft PR, it's still a work in progress as I state in the description but there's quite a bit here so feel free...
Looks like #2796 was merged so I will rebase and refactor this PR based on that when I get back to this. I'm planning to continue working on this PR...
@EdColeman and @dlmarion - Ok I have updated my PR with a few changes to address the comments/discussion made as well as added/changed some tests. I rebased my PR off...
@EdColeman - I just rebased off main since I saw you merged in your PropUtil changes and pushed up the latest
@ctubbsii, @EdColeman, @dlmarion - I just pushed an updated draft PR that has changes to handle detect and handle version changes when modifying properties and is ready for review again....
@ctubbsii - This PR has been updated to remove the retry on ConcurrentModificationException
I went ahead and squashed my commits into 1 (leaving Emily's) and I also rebased and removed the draft status as I think this PR is about ready to merge...
> > I went ahead and squashed my commits into 1 (leaving Emily's) and I also rebased and removed the draft status as I think this PR is about ready...
@Manno15 - I went ahead and restored my previous commit to make it easier to review the concurrent modification changes so the PR is back to 3 commits again.
@EdColeman - Here is my first attempt at renaming the methods and using an enum. Per our discussion offline I renamed the "debug" property to "target" as it's a better...