I think refactored code can reduce code redundancy
Comment From: hazendaz
seems mostly good but what about the hash, equals, etc you removed but did not add to the main object to process with the delegate. That may in fact be important especially to the cache. Not sure if the downstream cache projects are in fact calling those. They could be so I think you need to add them just to be safe since this is a fairly big change.
Comment From: harawata
Thank you for the PR, @causeThenEffect !
I haven't looked into it, but there are test failures.
Comment From: wuwen5
This appears to be a duplicate of #838. If necessary, I can reopen it.
After rebase, it's here https://github.com/wuwen5/mybatis-3/commit/a795b01998e674658dbc166dd547826079d4d7f7
Comment From: hazendaz
@wuwen5 Go ahead and reopen. If it is just a rehash, no one said anything the first time through, we should be able to compare them. Main thing is it has to build successfully and address any concerns.