makes me wonder if we have unnecessary code duplication between executeInserts and executeUpdates - but we should not fix that in this PR, it would be a larger refactoring. but could we pass the node to processAssoc and let that method update the node as well? that would reduce the duplication