You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by "Velmurugan Periasamy (JIRA)" <ji...@apache.org> on 2015/11/13 19:15:11 UTC
[jira] [Created] (RANGER-733) Implement best coding practices to
resolve issues found during code scan
Velmurugan Periasamy created RANGER-733:
-------------------------------------------
Summary: Implement best coding practices to resolve issues found during code scan
Key: RANGER-733
URL: https://issues.apache.org/jira/browse/RANGER-733
Project: Ranger
Issue Type: Bug
Affects Versions: 0.6.0
Reporter: Velmurugan Periasamy
Assignee: Velmurugan Periasamy
Fix For: 0.6.0
*+Fix reported issues in below classes:+*
+File: org/apache/ranger/json/JsonDateSerializer.java+
Call to method of static java.text.DateFormat in org.apache.ranger.json.JsonDateSerializer.serialize(Date, JsonGenerator, SerializerProvider)
+Method: org.apache.ranger.biz.XUserMgr.updateXModuleDefPermission(VXModuleDef)+
Make sure that getIsAllowed() is done without AUTOBOXING
+File: org/apache/ranger/service/XAccessAuditService.java+
Field XAccessAuditService.searchFields masks field in superclass org.apache.ranger.service.AbstractBaseResourceService
Field XAccessAuditService.sortFields masks field in superclass org.apache.ranger.service.AbstractBaseResourceService
+File: org/apache/ranger/service/XPolicyService.java+
Line#: 239
if (assetType == 0 || assetType == AppConstants.ASSET_UNKNOWN) - are same condition!
+File: org/apache/ranger/view/VXGroupPermission.java+
Field VXGroupPermission.id masks field in superclass org.apache.ranger.view.VXDataObject
+File: org/apache/ranger/view/VXModuleDef.java+
Field VXModuleDef.id masks field in superclass org.apache.ranger.view.VXDataObject
+File: org/apache/ranger/view/VXUserPermission.java+
Field VXUserPermission.id masks field in superclass org.apache.ranger.view.VXDataObject
*+Fix logging in below classes:+*
./unixauthclient/src/main/java/org/apache/ranger/authentication/unix/jaas/RemoteUnixLoginModule.java
./lookup-client/src/main/java/org/apache/ranger/storm/client/StormClient.java
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)