You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Illya Yalovyy <ya...@amazon.com> on 2016/10/03 17:38:33 UTC
Review Request 52487: HIVE-14875 Enhancement and refactoring of
TestLdapAtnProviderWithMiniDS
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52487/
-----------------------------------------------------------
Review request for hive, Ashutosh Chauhan, Chaoyu Tang, and Szehon Ho.
Repository: hive-git
Description
-------
This makes the following enhancements to defining the ldifs to apply
to the in-memory LDAP server:
* Extract defined ldifs to a resource file
* Remove unneeded attributes defined in each ldif entry such as:
* sn (Surname) and givenName from group entries
* distinguishedName from all entries as this attribute serves more
as a parent type of many other attributes.
* Remove setting ExtensibleObject as an objectClass for all ldap entries
as that is not needed. This objectClass would allow for adding any
attribute to an entry.
* Add missing uid attribute to group entries whose dn refer to a uid
attribute
* Add missing uidObject objectClass to entries that have the uid attribute
* Explicitly set organizationalPerson objectClass to user entries as
they are using inetOrgPerson objectClass which is a subclass of
the organizationalPerson objectClass
* Create indexes on cn and uid attributes as they are commonly
queried.
* Removed unused variables and imports.
* Fixed givenName for user3.
* Other minor code clean up.
Diffs
-----
service/src/test/org/apache/hive/service/auth/TestLdapAtnProviderWithMiniDS.java 23a048a
service/src/test/resources/ldap/example.com.ldif PRE-CREATION
Diff: https://reviews.apache.org/r/52487/diff/
Testing
-------
Tests run: 126, Failures: 0, Errors: 0, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 04:00 min
[INFO] Finished at: 2016-09-29T21:20:04-07:00
[INFO] Final Memory: 40M/1000M
[INFO] ------------------------------------------------------------------------
Thanks,
Illya Yalovyy
Re: Review Request 52487: HIVE-14875 Enhancement and refactoring of
TestLdapAtnProviderWithMiniDS
Posted by Illya Yalovyy <ya...@amazon.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52487/
-----------------------------------------------------------
(Updated Oct. 3, 2016, 5:40 p.m.)
Review request for hive, Ashutosh Chauhan, Chaoyu Tang, and Szehon Ho.
Repository: hive-git
Description (updated)
-------
This makes the following enhancements to defining the ldifs to apply
to the in-memory LDAP server:
* Extract defined ldifs to a resource file
* Remove unneeded attributes defined in each ldif entry such as:
* sn (Surname) and givenName from group entries
* distinguishedName from all entries as this attribute serves more
as a parent type of many other attributes.
* Remove setting ExtensibleObject as an objectClass for all ldap entries
as that is not needed. This objectClass would allow for adding any
attribute to an entry.
* Add missing uid attribute to group entries whose dn refer to a uid
attribute
* Add missing uidObject objectClass to entries that have the uid attribute
* Explicitly set organizationalPerson objectClass to user entries as
they are using inetOrgPerson objectClass which is a subclass of
the organizationalPerson objectClass
* Create indexes on cn and uid attributes as they are commonly
queried.
* Removed unused variables and imports.
* Fixed givenName for user3.
* Other minor code clean up.
https://issues.apache.org/jira/browse/HIVE-14875
Diffs
-----
service/src/test/org/apache/hive/service/auth/TestLdapAtnProviderWithMiniDS.java 23a048a
service/src/test/resources/ldap/example.com.ldif PRE-CREATION
Diff: https://reviews.apache.org/r/52487/diff/
Testing
-------
Tests run: 126, Failures: 0, Errors: 0, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 04:00 min
[INFO] Finished at: 2016-09-29T21:20:04-07:00
[INFO] Final Memory: 40M/1000M
[INFO] ------------------------------------------------------------------------
Thanks,
Illya Yalovyy