----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24763/#review50917 -----------------------------------------------------------
Ship it! Ship It! - Brock Noland On Aug. 17, 2014, 10 p.m., Venki Korukanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24763/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2014, 10 p.m.) > > > Review request for hive, Brock Noland and Szehon Ho. > > > Repository: hive-git > > > Description > ------- > > Please see JIRA HIVE-7606 for description > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkTask.java 5ac5a25 > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/session/SparkSession.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/session/SparkSessionImpl.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/session/SparkSessionManager.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/session/SparkSessionManagerImpl.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 9798cf3 > > ql/src/test/org/apache/hadoop/hive/ql/exec/spark/session/TestSparkSessionManagerImpl.java > PRE-CREATION > service/src/java/org/apache/hive/service/server/HiveServer2.java 0864dfb > > Diff: https://reviews.apache.org/r/24763/diff/ > > > Testing > ------- > > Added unittests to test SessionManagerImpl in single session mode (HiveCLI) > and multi-session mode (HiveServer2). Also tested few queries from Hive CLI. > Testing using actual HiveServer2 is blocked due to HIVE-7747. > > > Thanks, > > Venki Korukanti > >
