svn commit: r1586110 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java

[om...@apache.org]

Author: omalley
Date: Wed Apr  9 19:54:13 2014
New Revision: 1586110

URL: http://svn.apache.org/r1586110
Log:
HIVE-6818. I forgot to add the comment that Sergey wanted to add as part of the commit.

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java?rev=1586110&r1=1586109&r2=1586110&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java Wed Apr  9 19:54:13 2014
@@ -166,7 +166,12 @@ public interface SearchArgument {
   public TruthValue evaluate(TruthValue[] leaves);
 
   /**
-   * Serialize the SARG as a kyro object and return the base64 strig.
+   * Serialize the SARG as a kyro object and return the base64 string.
+   *
+   * Hive should replace the current XML-based AST serialization for predicate pushdown
+   * with the Kryo serialization of the SARG because the representation is much more
+   * compact and focused on what is needed for predicate pushdown.
+   *
    * @return the serialized SARG
    */
   public String toKryo();