You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Min Zhou <co...@gmail.com> on 2009/07/20 07:39:41 UTC

It's strange two if clause with the same logics

Hi,
see ObjectInspectorConverters.java:78
          case STRING:
            if (outputOI instanceof WritableStringObjectInspector) {
              return new PrimitiveObjectInspectorConverter.TextConverter(
                  (PrimitiveObjectInspector)inputOI);
            } else if  (outputOI instanceof WritableStringObjectInspector) {
              return new PrimitiveObjectInspectorConverter.TextConverter(
                  (PrimitiveObjectInspector)inputOI);
            }

the second clause has the same logic with the first one. I guess there is
something wrong, maybe one is for WritableStringObjectInspector, the other
is for JavaStringObjectInspector.

Min
-- 
My research interests are distributed systems, parallel computing and
bytecode based virtual machine.

My profile:
http://www.linkedin.com/in/coderplay
My blog:
http://coderplay.javaeye.com