Hi Zizhuan, Thanks for your careful review!
> 5. A small suggestion > After further consideration, I suggest using the condition: > if (RelationIsPopulated(matviewRel) != !skipData) > This is much more readable and intuitive compared to the original: > if (RelationIsPopulated(matviewRel) == skipData) > Especially when paired with the original call: > SetMatViewPopulatedState(matviewRel, !skipData); Both are ok for me. -- Regards, ChangAo Chen
