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

[omalley]

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();