I'm not sure if it's still correct, but based on my understanding, OptimizedHybridHashJoinOperatorDescriptor does the role reversal optimization which was done by Pouria, while HybridHashJoinOperatorDescriptor was the old implementation before Pouria's work and probably could be deleted.
Best, Yingyi On Thu, Nov 8, 2018 at 6:52 PM Xikui Wang <[email protected]> wrote: > Hi Devs, > > Does anyone know what's the difference between > the OptimizedHybridHashJoinOperatorDescriptor and the > HybridHashJoinOperatorDescriptor? > > I was going over part of the join codes and found both of these classes > there. It seems the only difference between the two is when there is a > hashFunctionFamily is null, we will use the > HybridHashJoinOperatorDescriptor. However, it seems this would not > happen... In fact, I changed to code to always use the optimized one and no > test cases fail (could be test cases bias though). Thanks ahead! > > Best, > Xikui >
