It's strange two if clause with the same logics

[Min Zhou]

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