right, but one type check is not excessive and bucket_type can be a string OR an BucketType object in other parts of the code. It just seems confusing.