Good point. Updated.


On Mon, Jan 28, 2013 at 11:15 AM, Raghu Angadi <notifications@github.com>wrote:

> In pig/src/main/java/com/twitter/elephantbird/pig/util/ThriftToPig.java:
>
> > @@ -79,6 +84,17 @@ public Tuple getLazyTuple(M thriftObj) {
> >      return new LazyTuple(structDesc, thriftObj);
> >    }
> >
> > +  /**
> > +   * Set the flags that can be used by the conversion.
> > +   * @param conf usually the Hadoop job conf
> > +   */
> > +  public static void setConversionProperties(Configuration conf) {
> > +    if (conf != null) {
> > +      useEnumId = conf.getBoolean(USE_ENUM_ID_CONF_KEY, false);
> > +      LOG.info("useEnumId is set to " + useEnumId);
>
> can you make this a debug log? It would be easy to see if this config does
> not take affect. Pig creates multiple of these loaders and we will see lots
> of these in the log.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/kevinweil/elephant-bird/pull/284/files#r2797635>.
>
>