You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by cw...@apache.org on 2019/05/23 20:41:11 UTC

[incubator-druid] 01/01: [Backport] OrcStructConvert fix

This is an automated email from the ASF dual-hosted git repository.

cwylie pushed a commit to branch clintropolis-patch-1
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git

commit 3100c61f1220f6593d160818719a416e6ca53064
Author: Clint Wylie <cw...@apache.org>
AuthorDate: Thu May 23 13:40:59 2019 -0700

    [Backport] OrcStructConvert fix
    
    from #7738, this is a performance issue
---
 .../main/java/org/apache/druid/data/input/orc/OrcStructConverter.java   | 2 --
 1 file changed, 2 deletions(-)

diff --git a/extensions-core/orc-extensions/src/main/java/org/apache/druid/data/input/orc/OrcStructConverter.java b/extensions-core/orc-extensions/src/main/java/org/apache/druid/data/input/orc/OrcStructConverter.java
index 20fbf06..69d113d 100644
--- a/extensions-core/orc-extensions/src/main/java/org/apache/druid/data/input/orc/OrcStructConverter.java
+++ b/extensions-core/orc-extensions/src/main/java/org/apache/druid/data/input/orc/OrcStructConverter.java
@@ -166,8 +166,6 @@ public class OrcStructConverter
         fieldIndexCache.put(fields.get(i), i);
       }
     }
-    WritableComparable wc = struct.getFieldValue(fieldName);
-
     int fieldIndex = fieldIndexCache.getOrDefault(fieldName, -1);
 
     return convertField(struct, fieldIndex);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org