You should keep this `private[spark]`, we can still use it in sub packages. Also, why did you need to remove the `>: Null`? I'll CC @rxin to see if this can cause any problems. It might mean we don't get a reasonable "no value" indicator here, so perhaps we should keep the `>: Null` and change the case where this was not working.