> On Feb. 20, 2017, 9:46 a.m., Colm O hEigeartaigh wrote: > > Thanks for making the changes. Could you update the patch to the latest > > master, it doesn't apply as there was a change to AtlasClient. The other > > change I'd like to see is that there is still a "atlasResourceParamter" > > (instead of atlasResourceParameter) in AtlasResourceMgr. > > Colm O hEigeartaigh wrote: > *bump* Please address my last comment and I'll review again with a view > to merging the fix. > > Qiang Zhang wrote: > Ok. I will update the patch after carefully tested and verified. Thanks!
I had modified code according to your review after updated lasted source from ranger github repository. Other contributor modified related class file.(https://github.com/apache/ranger/blob/master/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasClient.java). I modified records as following: a) AtlasResourceMgr: There are no spaces around "=" here: "atlasResourceName=context.getResourceName();" Modified. b) AtlasResourceMgr: Change "atlasResourceParamter" to "atlasResourceParameter Modified. c) AtlasResourceMgr: There is an indentation problem in getAtlasResource() Modified. d) Are the changes to RangerAtlasAuthorizer + RangerAtlasResource required? We should limit whitespace fixing just to classes that are being changed as part of the commit. Modified. e) AtlasClient: Missing space after = "this.statusUrl =atlasUrl + ATLAS_STATUS_API_ENDPOINT;" Modified. f) AtlasClient: Change this to 'username': LOG.error("No value found for configuration 'usename'. Modified. g) AtlasClient: Indentation wrong on line 315 "return lret;" Modified. h) AtlasClient: Indentation wrong in the try block: lines 121 -> 135 Modified. i) AtlasClient: getStatusResponse() method should be private Modified. Please review the lasted patch. Thanks. - Qiang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56654/#review166073 ----------------------------------------------------------- On March 1, 2017, 9:03 a.m., Qiang Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56654/ > ----------------------------------------------------------- > > (Updated March 1, 2017, 9:03 a.m.) > > > Review request for ranger, Alok Lal, Don Bosco Durai, Colm O hEigeartaigh, > Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy. > > > Bugs: RANGER-1348 > https://issues.apache.org/jira/browse/RANGER-1348 > > > Repository: ranger > > > Description > ------- > > Autocompletion not working properly Resource Autocompletion fuction I have > submit the patch please review it. > > > Diffs > ----- > > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java > ba4d263 > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasClient.java > b959f97 > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasConnectionMgr.java > 84e24c5 > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java > 93b2662 > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/json/model/ResourceEntityResponse.java > PRE-CREATION > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/json/model/ResourceOperationResponse.java > PRE-CREATION > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/json/model/ResourceTaxonomyResponse.java > PRE-CREATION > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/json/model/ResourceTermResponse.java > PRE-CREATION > > plugin-atlas/src/main/java/org/apache/ranger/services/atlas/json/model/ResourceTypeResponse.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/56654/diff/4/ > > > Testing > ------- > > Tested it > > > Thanks, > > Qiang Zhang > >
