Code review comment for lp:~kaaveeacs/drizzle/privatized-members-of-NestedJoin

Revision history for this message
Monty Taylor (mordred) wrote :

Hi, and thanks for the patch!

There is a random "y" on a line by itself.

there are two getSjCorrTables instead of a getSjCorrTables and a setSjCorrTables

many of the setters should probably be taking their params by const reference

review: Needs Fixing

« Back to merge proposal