Yeah, I think it's cleaner to use Option#isDefined/isEmpty...this type of `null` pattern appears in other parts too, such as for TachyonWriter and TablePartitionBuilder construction in MemoryStoreOperator. I'll fix that in another patch that will try to decouple RDD[TablePartition] construction and materialization from MemoryStoreSinkOperator.