> On 五月 4, 2021, 4:10 p.m., Abhay Kulkarni wrote:
> > intg/src/main/java/org/apache/ranger/RangerClient.java
> > Lines 504 (patched)
> > <https://reviews.apache.org/r/73326/diff/1/?file=2249696#file2249696line511>
> >
> > How is this method different from one on line 484? Can the earlier
> > method be used generically everwhere the class-type needs to be specified
> > (and this method be removed)?
this method return nothing, a sample is
```
public void deleteServiceDef(long serviceDefId) throws RangerServiceException {
callAPI(DELETE_SERVICEDEF_BY_ID.applyUrlFormat(serviceDefId), null);
}
```
If use
```
callAPI(DELETE_SERVICEDEF_BY_ID.applyUrlFormat(serviceDefId), null);
```
will got e error
```
Ambiguous method call. Both
callAPI(API,Map<String, String>, Object, Class<Object>)
callAPI(API,Map<String, String>,Object,GenericType<Object>)
in RangerClient match
```
- Zhou
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73326/#review222944
-----------------------------------------------------------
On 四月 30, 2021, 8:53 p.m., Abhishek Kumar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73326/
> -----------------------------------------------------------
>
> (Updated 四月 30, 2021, 8:53 p.m.)
>
>
> Review request for ranger, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani,
> Sailaja Polavarapu, and Velmurugan Periasamy.
>
>
> Bugs: RANGER-3227
> https://issues.apache.org/jira/browse/RANGER-3227
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Fix deserialization issues when parsing REST reponses by using GenericType.
>
>
> Diffs
> -----
>
> intg/src/main/java/org/apache/ranger/RangerClient.java c7e32dc59
>
> ranger-examples/sample-client/src/main/java/org/apache/ranger/examples/sampleclient/SampleClient.java
> b380127bf
>
>
> Diff: https://reviews.apache.org/r/73326/diff/1/
>
>
> Testing
> -------
>
> Tested the changes on local machine.
>
>
> Thanks,
>
> Abhishek Kumar
>
>