A. Sophie Blee-Goldman
A. Sophie Blee-Goldman
> Wow every single test passed. Don't see that very often on a pr Yeah that is legitimately amazing. Love to see it! Maybe things are turning around...heh... š
Ping @cadonna since he's got a good eye for this part of the code. Also @wcarlson5 @lct45 @showuon for help with reviews
Yeah, any change to the public interface is going to require a KIP. And tbh the state store interface is already messy enough, I think we should try to avoid...
@vvcephei @cadonna @mjsax @guozhangwang anyone have any thoughts on this?
cc also @cadonna who's familiar with these specific tests I think
I think this looks good but there's a merge conflict, can you fix that?
Test failures are unrelated, merged to trunk. Thanks for the fix!
FYI I cherrypicked this back to 3.7 while cherrypicking back another sticky assignor fix. Should be in the 3.7.1 release
@flashmouse can you rebase this PR please?
Test failures are unrelated, merging to trunk