It's confusing to pass m_partitioningColumns to addPartitioningColumns -- it's available as a data member there, too.