Yea that's exactly what I did.  It's totally not needed.
I probably never looked at it again after it was working.

----- David Cuddeback's Original Message -----
> > +  UnionNamePartitioner
> > +  [type]
> > +
> > +
> > +  p/VerticalPartitioner
> > +  (p/make-partition
> > +    [this object]
> > +    (vector (union/current-field-name object)))
> > +
> > +
> > +  (p/validate
> > +    [this dirs]
> > +    [(try
> > +       (contains? (type/field-names type)
> > +                  (first dirs))
> > +       (catch Exception e
> 
> Is it necessary to catch an exception here? I think you copied the code structure from the other partitioner, which catch `NumberFormatException` because it needed to parse integers. I don't see how this code could throw an exception.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dcuddeback/pail-thrift/pull/1/files#r8780202