----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48502/#review136949 -----------------------------------------------------------
itests/hive-unit/src/test/java/org/apache/hadoop/hive/llap/ext/TestLlapInputSplit.java (line 85) <https://reviews.apache.org/r/48502/#comment202058> Should we also be checking the token bytes here? llap-ext-client/src/java/org/apache/hadoop/hive/llap/LlapBaseInputFormat.java (line 135) <https://reviews.apache.org/r/48502/#comment202059> Looks like a null tokenBytes is serialized/deserialized as a 0-byte array. During token verification does this get treated the same way as a null token? - Jason Dere On June 9, 2016, 7:07 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48502/ > ----------------------------------------------------------- > > (Updated June 9, 2016, 7:07 p.m.) > > > Review request for hive, Jason Dere and Siddharth Seth. > > > Repository: hive-git > > > Description > ------- > > . > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/llap/ext/TestLlapInputSplit.java > 7491222 > llap-client/src/java/org/apache/hadoop/hive/llap/LlapInputSplit.java > ab11926 > > llap-client/src/java/org/apache/hadoop/hive/llap/ext/LlapTaskUmbilicalExternalClient.java > 3ebae4a > > llap-ext-client/src/java/org/apache/hadoop/hive/llap/LlapBaseInputFormat.java > d1748cb > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java > cff5ee1 > > Diff: https://reviews.apache.org/r/48502/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >