galderz/infinispan — comment on commit fa419a1b22f56faa49e66588728fa9d580aafaed
File: core/src/main/java/org/infinispan/interceptors/CacheLoaderInterceptor.java line 151
By Sanne:

I'm confused on this one. Since the fix above in AbstractInvocationContextContainer is overriding the context with the InvocationContextFlagsOverride to copy the flags in the context, shouldn't we stay consistent and read the flags out of the context instead of the command?

Changing this to read only from the command - knowing that the flags are copied in the context - makes me think that we might miss flags contained in the context. Also I think we should pick one strategy and stay consistent: where do you suggest we read flags from? Always from the command? If so, should we delete InvocationContextFlagsOverride ?