----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71668/#review218546 -----------------------------------------------------------
agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json Lines 4 (patched) <https://reviews.apache.org/r/71668/#comment306280> "Hadoop SQL Query Engine" => "Hadoop SQL" security-admin/db/mysql/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql Lines 22 (patched) <https://reviews.apache.org/r/71668/#comment306286> Can the value of the new column 'display_name' be set to exisitng value in column 'name'? If yes, we can avoid the Java patch to populate the new column for x_service_def and x_service. - Madhan Neethiraj On Nov. 6, 2019, 12:38 p.m., Kishor Gollapalliwar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71668/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2019, 12:38 p.m.) > > > Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan > Neethiraj, Mehul Parikh, Pradeep Agrawal, and Velmurugan Periasamy. > > > Bugs: RANGER-2590 > https://issues.apache.org/jira/browse/RANGER-2590 > > > Repository: ranger > > > Description > ------- > > Name of service-defs (hdfs/hive/hbase/kafka/..) and services > (test_hive/prod_hive/..) are rendered in Ranger landing page. It will be > helpful to see a more descriptive string in the UI - like Apache Kafka, > Apache HBase, Production Hive, Test Hive. To support this: > > service-def and service should be enhanced with an additional attribute > displayName > UI should be updated to use the value in displayName instead of name > default value of displayName should be the value of name > displayName should be unique - just as name is. > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/errors/ValidationErrorCode.java > 2927362ab > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerService.java > ed8d38417 > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java > 50d84a436 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefHelper.java > 0954beb82 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefValidator.java > f96fcfc92 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceValidator.java > 9b5d8d511 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidator.java > c4ec63b95 > > agents-common/src/main/java/org/apache/ranger/plugin/store/ServiceStore.java > 4af457e5c > agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java > 93b28a82e > agents-common/src/main/resources/service-defs/ranger-servicedef-abfs.json > 18454deff > agents-common/src/main/resources/service-defs/ranger-servicedef-atlas.json > 07a980031 > > agents-common/src/main/resources/service-defs/ranger-servicedef-elasticsearch.json > b055bf818 > agents-common/src/main/resources/service-defs/ranger-servicedef-hbase.json > 71fae66d4 > agents-common/src/main/resources/service-defs/ranger-servicedef-hdfs.json > 2e5d07c2f > agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json > 5722e0948 > agents-common/src/main/resources/service-defs/ranger-servicedef-kafka.json > 800b12367 > agents-common/src/main/resources/service-defs/ranger-servicedef-kms.json > f96cb9cd1 > agents-common/src/main/resources/service-defs/ranger-servicedef-knox.json > 495a69913 > agents-common/src/main/resources/service-defs/ranger-servicedef-kylin.json > a6e76a01c > > agents-common/src/main/resources/service-defs/ranger-servicedef-nifi-registry.json > 8235edc3d > agents-common/src/main/resources/service-defs/ranger-servicedef-nifi.json > 5f1e46f09 > agents-common/src/main/resources/service-defs/ranger-servicedef-ozone.json > 5787d93d6 > agents-common/src/main/resources/service-defs/ranger-servicedef-presto.json > 151c7d2fa > agents-common/src/main/resources/service-defs/ranger-servicedef-solr.json > 2f12721e1 > agents-common/src/main/resources/service-defs/ranger-servicedef-sqoop.json > 8cff9abdd > agents-common/src/main/resources/service-defs/ranger-servicedef-storm.json > 03c1574ff > agents-common/src/main/resources/service-defs/ranger-servicedef-tag.json > c17b75070 > agents-common/src/main/resources/service-defs/ranger-servicedef-wasb.json > 946b662ad > agents-common/src/main/resources/service-defs/ranger-servicedef-yarn.json > a32c08d93 > > agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerServiceValidator.java > c271dd955 > security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql > 1857a77e9 > > security-admin/db/mysql/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql > PRE-CREATION > > security-admin/db/mysql/patches/045-add-otherAttributes-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql > 0293abeaa > > security-admin/db/oracle/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql > PRE-CREATION > > security-admin/db/oracle/patches/045-add-otherAttributes-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql > 1d1a31c51 > > security-admin/db/postgres/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql > PRE-CREATION > > security-admin/db/postgres/patches/045-add-otherAttributes-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > > security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql > 5381398da > > security-admin/db/sqlanywhere/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql > PRE-CREATION > > security-admin/db/sqlanywhere/patches/045-add-otherAttributes-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql > d24de6863 > > security-admin/db/sqlserver/patches/044-add-displayName-col-in-x_service_def_and_x_service.sql > PRE-CREATION > > security-admin/db/sqlserver/patches/045-add-otherAttributes-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > 866eed98b > security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java > 99af81804 > security-admin/src/main/java/org/apache/ranger/db/XXServiceDao.java > 0791f2f54 > security-admin/src/main/java/org/apache/ranger/db/XXServiceDefDao.java > beef5bf49 > security-admin/src/main/java/org/apache/ranger/entity/XXGroup.java > dc95b58c0 > security-admin/src/main/java/org/apache/ranger/entity/XXPortalUser.java > 89b180db4 > security-admin/src/main/java/org/apache/ranger/entity/XXServiceBase.java > 366dc5567 > security-admin/src/main/java/org/apache/ranger/entity/XXServiceDefBase.java > 9a353599a > security-admin/src/main/java/org/apache/ranger/entity/XXUser.java 47200951a > > security-admin/src/main/java/org/apache/ranger/patch/PatchForXServiceUpdate_J10031.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > d60b73809 > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceDefServiceBase.java > 9b543ef8d > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java > 22d3c60da > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceServiceBase.java > 7db67ac13 > > security-admin/src/main/java/org/apache/ranger/service/XGroupServiceBase.java > 378e3933c > > security-admin/src/main/java/org/apache/ranger/service/XPortalUserServiceBase.java > 1c0b7c8b3 > > security-admin/src/main/java/org/apache/ranger/service/XUserServiceBase.java > 9669c0b4f > security-admin/src/main/java/org/apache/ranger/view/VXGroup.java 624dd5ad2 > security-admin/src/main/java/org/apache/ranger/view/VXPortalUser.java > 02956494a > security-admin/src/main/java/org/apache/ranger/view/VXUser.java 1c012191f > security-admin/src/main/resources/META-INF/jpa_named_queries.xml b619b7a92 > security-admin/src/main/webapp/scripts/models/RangerServiceDef.js 398941abd > > security-admin/src/main/webapp/scripts/views/service/RangerServiceViewDetail.js > a169baa7c > security-admin/src/main/webapp/scripts/views/service/ServiceForm.js > b7215f2c1 > > security-admin/src/main/webapp/templates/common/ServiceManagerLayout_tmpl.html > 82b667ccf > security-admin/src/main/webapp/templates/helpers/XAHelpers.js 4b64d01b5 > > security-admin/src/main/webapp/templates/service/RangerServiceViewDetail_tmpl.html > d9fe51f1b > security-admin/src/test/java/org/apache/ranger/rest/TestServiceREST.java > f687ab8bc > > > Diff: https://reviews.apache.org/r/71668/diff/3/ > > > Testing > ------- > > Build succeeded on apache master > Run setup using above build > Validated following scenarios for ranger-admin > Service : > Create with empty displayName, displayName = name > Create with valid displayName, displayName = displayName > Create with existing displayName, creation failed > Update with empty displayName, displayName = name > Update with valid displayName, displayName = displayName > Update both name and displayName, both updated succcessfully. > Update with existing displayName, update failed > > ServiceDef : > Create with empty displayName, displayName = name > Create with valid displayName, displayName = displayName > Update with empty displayName, displayName = name > Update with valid displayName, displayName = displayName > Update both name and displayName, both updated succcessfully. > > > Thanks, > > Kishor Gollapalliwar > >
