hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thejas Nair" <the...@hortonworks.com>
Subject Re: Review Request 24167: HIVE-7533 : sql std auth - set authorization privileges for tables when created from hive cli
Date Mon, 04 Aug 2014 07:32:23 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24167/
-----------------------------------------------------------

(Updated Aug. 4, 2014, 7:32 a.m.)


Review request for hive and Jason Dere.


Changes
-------

fixing formatting issue in HIveConf


Bugs: HIVE-7533
    https://issues.apache.org/jira/browse/HIVE-7533


Repository: hive-git


Description
-------

See jira https://issues.apache.org/jira/browse/HIVE-7533


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 15bc0a3 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/authorization/plugin/TestHiveAuthorizerCheckInvocation.java
c91b15c 
  itests/hive-unit/src/test/java/org/apache/hive/jdbc/authorization/TestCLIAuthzSessionContext.java
PRE-CREATION 
  itests/hive-unit/src/test/java/org/apache/hive/jdbc/authorization/TestHS2AuthzContext.java
f42390c 
  itests/hive-unit/src/test/java/org/apache/hive/jdbc/authorization/TestHS2AuthzSessionContext.java
PRE-CREATION 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessControllerForTest.java
bdbe8a7 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizerFactoryForTest.java
3395513 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 8f3f2b6 
  ql/src/java/org/apache/hadoop/hive/ql/processors/CommandUtil.java 87c8fbb 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerFactory.java
876527a 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthzContext.java
bb71a7e 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthzSessionContext.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java
140882d 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizerFactory.java
c2d50b1 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java fcfcf42 
  ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/TestSessionUserName.java 273c4b9

  ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessController.java
226b16e 
  ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerCLI.java
PRE-CREATION 
  ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
PRE-CREATION 
  ql/src/test/queries/clientnegative/authorization_addjar.q aad703c 
  ql/src/test/queries/clientnegative/authorization_addpartition.q 8abdd2b 
  ql/src/test/queries/clientnegative/authorization_alter_db_owner.q f716262 
  ql/src/test/queries/clientnegative/authorization_alter_db_owner_default.q f904935 
  ql/src/test/queries/clientnegative/authorization_cannot_create_all_role.q 9bbd655 
  ql/src/test/queries/clientnegative/authorization_cannot_create_default_role.q 32045e4 
  ql/src/test/queries/clientnegative/authorization_cannot_create_none_role.q 42369d8 
  ql/src/test/queries/clientnegative/authorization_caseinsensitivity.q 7b47b65 
  ql/src/test/queries/clientnegative/authorization_cli_auth_enable.q PRE-CREATION 
  ql/src/test/queries/clientnegative/authorization_compile.q 857306b 
  ql/src/test/queries/clientnegative/authorization_create_func1.q 02bbe09 
  ql/src/test/queries/clientnegative/authorization_create_func2.q 8760fa8 
  ql/src/test/queries/clientnegative/authorization_create_macro1.q c904a10 
  ql/src/test/queries/clientnegative/authorization_create_role_no_admin.q a84fe64 
  ql/src/test/queries/clientnegative/authorization_createview.q 9b1f2ea 
  ql/src/test/queries/clientnegative/authorization_ctas.q 1cf74a3 
  ql/src/test/queries/clientnegative/authorization_deletejar.q f2e95d2 
  ql/src/test/queries/clientnegative/authorization_desc_table_nosel.q 47663c9 
  ql/src/test/queries/clientnegative/authorization_dfs.q 9dc8a64 
  ql/src/test/queries/clientnegative/authorization_disallow_transform.q 64b300c 
  ql/src/test/queries/clientnegative/authorization_drop_db_cascade.q 78ec078 
  ql/src/test/queries/clientnegative/authorization_drop_db_empty.q 993a2a3 
  ql/src/test/queries/clientnegative/authorization_drop_role_no_admin.q 2593e9e 
  ql/src/test/queries/clientnegative/authorization_droppartition.q f05e945 
  ql/src/test/queries/clientnegative/authorization_fail_8.q 37213a2 
  ql/src/test/queries/clientnegative/authorization_grant_table_allpriv.q f3c86b9 
  ql/src/test/queries/clientnegative/authorization_grant_table_dup.q 7808cb3 
  ql/src/test/queries/clientnegative/authorization_grant_table_fail1.q 8dc8e45 
  ql/src/test/queries/clientnegative/authorization_grant_table_fail_nogrant.q d51c1c3 
  ql/src/test/queries/clientnegative/authorization_insert_noinspriv.q 2fa3cb2 
  ql/src/test/queries/clientnegative/authorization_insert_noselectpriv.q b9bee4e 
  ql/src/test/queries/clientnegative/authorization_insertoverwrite_nodel.q 73cfe82 
  ql/src/test/queries/clientnegative/authorization_invalid_priv_v2.q 9c72408 
  ql/src/test/queries/clientnegative/authorization_not_owner_alter_tab_rename.q 8a3300c 
  ql/src/test/queries/clientnegative/authorization_not_owner_alter_tab_serdeprop.q 0172c4c

  ql/src/test/queries/clientnegative/authorization_not_owner_drop_tab.q 2d0e52d 
  ql/src/test/queries/clientnegative/authorization_not_owner_drop_view.q 76bbab4 
  ql/src/test/queries/clientnegative/authorization_priv_current_role_neg.q 29a06a8 
  ql/src/test/queries/clientnegative/authorization_revoke_table_fail1.q e19bf37 
  ql/src/test/queries/clientnegative/authorization_revoke_table_fail2.q 4b0cf32 
  ql/src/test/queries/clientnegative/authorization_role_cycles1.q fa3fc19 
  ql/src/test/queries/clientnegative/authorization_role_cycles2.q 069a663 
  ql/src/test/queries/clientnegative/authorization_role_grant.q 3f1a7b8 
  ql/src/test/queries/clientnegative/authorization_role_grant2.q 7d439cb 
  ql/src/test/queries/clientnegative/authorization_role_grant_nosuchrole.q 51ca345 
  ql/src/test/queries/clientnegative/authorization_role_grant_otherrole.q 9440c45 
  ql/src/test/queries/clientnegative/authorization_role_grant_otheruser.q 1db2806 
  ql/src/test/queries/clientnegative/authorization_rolehierarchy_privs.q 8bb776e 
  ql/src/test/queries/clientnegative/authorization_select.q 3987179 
  ql/src/test/queries/clientnegative/authorization_select_view.q a4071cd 
  ql/src/test/queries/clientnegative/authorization_set_role_neg1.q 9ba3a82 
  ql/src/test/queries/clientnegative/authorization_set_role_neg2.q a810f1a 
  ql/src/test/queries/clientnegative/authorization_show_grant_otherrole.q e131766 
  ql/src/test/queries/clientnegative/authorization_show_grant_otheruser_all.q 4ef501d 
  ql/src/test/queries/clientnegative/authorization_show_grant_otheruser_alltabs.q 8cfc790

  ql/src/test/queries/clientnegative/authorization_show_grant_otheruser_wtab.q 4c3892c 
  ql/src/test/queries/clientnegative/authorization_show_parts_nosel.q d8190de 
  ql/src/test/queries/clientnegative/authorization_show_role_principals_no_admin.q 2afe87f

  ql/src/test/queries/clientnegative/authorization_show_roles_no_admin.q 0fc9fca 
  ql/src/test/queries/clientnegative/authorization_table_grant_nosuchrole.q 2065093 
  ql/src/test/queries/clientnegative/authorization_truncate.q 285600b 
  ql/src/test/queries/clientnegative/authorization_uri_add_partition.q d82ac71 
  ql/src/test/queries/clientnegative/authorization_uri_alterpart_loc.q d38ba74 
  ql/src/test/queries/clientnegative/authorization_uri_altertab_setloc.q c446b86 
  ql/src/test/queries/clientnegative/authorization_uri_create_table1.q c8e1fb4 
  ql/src/test/queries/clientnegative/authorization_uri_create_table_ext.q c8549b4 
  ql/src/test/queries/clientnegative/authorization_uri_createdb.q edfdf5a 
  ql/src/test/queries/clientnegative/authorization_uri_index.q 1a8f9cb 
  ql/src/test/queries/clientnegative/authorization_uri_insert.q 81b6e52 
  ql/src/test/queries/clientnegative/authorization_uri_insert_local.q 0a2fd89 
  ql/src/test/queries/clientnegative/authorization_uri_load_data.q 6af41f0 
  ql/src/test/queries/clientnegative/authorize_create_tbl.q d8beac3 
  ql/src/test/queries/clientnegative/temp_table_authorize_create_tbl.q bb75777 
  ql/src/test/queries/clientpositive/authorization_1_sql_std.q 34a9b68 
  ql/src/test/queries/clientpositive/authorization_admin_almighty1.q 380f506 
  ql/src/test/queries/clientpositive/authorization_admin_almighty2.q ba43e40 
  ql/src/test/queries/clientpositive/authorization_cli_createtab.q PRE-CREATION 
  ql/src/test/queries/clientpositive/authorization_create_func1.q 1f48deb 
  ql/src/test/queries/clientpositive/authorization_create_macro1.q 741fa92 
  ql/src/test/queries/clientpositive/authorization_create_table_owner_privs.q 17f4861 
  ql/src/test/queries/clientpositive/authorization_create_temp_table.q d49dbb4 
  ql/src/test/queries/clientpositive/authorization_grant_public_role.q 9da33dc 
  ql/src/test/queries/clientpositive/authorization_grant_table_priv.q 9c7a999 
  ql/src/test/queries/clientpositive/authorization_index.q 1f177ff 
  ql/src/test/queries/clientpositive/authorization_insert.q c1e4c1f 
  ql/src/test/queries/clientpositive/authorization_owner_actions.q 85d8b11 
  ql/src/test/queries/clientpositive/authorization_owner_actions_db.q 14a8d7c 
  ql/src/test/queries/clientpositive/authorization_parts.q bee091b 
  ql/src/test/queries/clientpositive/authorization_reset.q aa2c1fb 
  ql/src/test/queries/clientpositive/authorization_revoke_table_priv.q 1cd445b 
  ql/src/test/queries/clientpositive/authorization_role_grant1.q 1e8a6af 
  ql/src/test/queries/clientpositive/authorization_role_grant2.q d87d96f 
  ql/src/test/queries/clientpositive/authorization_set_show_current_role.q 4132370 
  ql/src/test/queries/clientpositive/authorization_show_grant.q bad1f07 
  ql/src/test/queries/clientpositive/authorization_view_sqlstd.q 91acd39 
  ql/src/test/results/clientnegative/authorization_cli_auth_enable.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/authorization_cli_createtab.q.out PRE-CREATION 

Diff: https://reviews.apache.org/r/24167/diff/


Testing
-------

New tests included.


Thanks,

Thejas Nair


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message