> On 5 6, 2016, 4:21 오후, Sergio Pena wrote: > > ql/src/java/org/apache/hadoop/hive/ql/session/YarnFairScheduling.java, line > > 24 > > <https://reviews.apache.org/r/47040/diff/1/?file=1373935#file1373935line24> > > > > There is another validation on Hadoop23Shims.refreshDefaultQueue(). Why > > don't we have that one too here?
I'm not sure what this means. - Reuben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47040/#review132054 ----------------------------------------------------------- On 5 5, 2016, 8:06 오후, Reuben Kuhnert wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47040/ > ----------------------------------------------------------- > > (Updated 5 5, 2016, 8:06 오후) > > > Review request for hive, Lenni Kuff, Mohit Sabharwal, and Sergio Pena. > > > Bugs: HIVE-13696 > https://issues.apache.org/jira/browse/HIVE-13696 > > > Repository: hive-git > > > Description > ------- > > Ensure that jobs sent to yarn with impersonation off are correctly routed to > the proper queue based on fair-scheduler.xml. Validate that jobs can only be > sent to queues authorized for the user. > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/Driver.java > 6a610cbcb1deb7f7f55bb8aff58020b057454b31 > ql/src/java/org/apache/hadoop/hive/ql/session/YarnFairScheduling.java > PRE-CREATION > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > a0015ebc655931f241b28c53fbb94cfe172841b1 > shims/common/src/main/java/org/apache/hadoop/fs/FileSystemWatcher.java > PRE-CREATION > shims/common/src/main/java/org/apache/hadoop/hive/shims/SchedulerShim.java > 63803b8b0752745bd2fedaccc5d100befd97093b > > shims/scheduler/src/main/java/org/apache/hadoop/hive/schshim/FairSchedulerShim.java > 372244dc3c989d2a3ae2eb2bfb8cd0a235705e18 > > Diff: https://reviews.apache.org/r/47040/diff/ > > > Testing > ------- > > > Thanks, > > Reuben Kuhnert > >