----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68630/#review208550 -----------------------------------------------------------
Ship it! Thanks for uploading a new patch. LGTM +1 - Laszlo Pinter On Sept. 11, 2018, 11:19 p.m., Daniel Dai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68630/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2018, 11:19 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > See HIVE-20420 > > > Diffs > ----- > > ql/pom.xml a55cbe3 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java > 12be41c > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/fallback/FallbackHiveAuthorizer.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/fallback/FallbackHiveAuthorizerFactory.java > PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_addjar.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_compile.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_create_func1.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_create_func2.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_dfs.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_disallow_transform.q > PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_load.q PRE-CREATION > ql/src/test/queries/clientnegative/fallbackauth_set_invalidconf.q > PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_addjar.q.out PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_compile.q.out PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_create_func1.q.out > PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_create_func2.q.out > PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_dfs.q.out PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_disallow_transform.q.out > PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_load.q.out PRE-CREATION > ql/src/test/results/clientnegative/fallbackauth_set_invalidconf.q.out > PRE-CREATION > > > Diff: https://reviews.apache.org/r/68630/diff/2/ > > > Testing > ------- > > > Thanks, > > Daniel Dai > >
