You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by zh...@zte.com.cn on 2016/10/15 07:14:56 UTC

Review Request 52907: change variable name

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/
-----------------------------------------------------------

Review request for ranger.


Bugs: RANGER-1183
    https://issues.apache.org/jira/browse/RANGER-1183


Repository: ranger


Description
-------

The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.


Diffs
-----

  plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 

Diff: https://reviews.apache.org/r/52907/diff/


Testing
-------


Thanks,

zhangqiang2


Re: Review Request 52907: Change variable name

Posted by Don Bosco Durai <bo...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/#review152797
-----------------------------------------------------------


Ship it!




Ship It!

- Don Bosco Durai


On Oct. 15, 2016, 7:22 a.m., zhangqiang2 wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52907/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2016, 7:22 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-1183
>     https://issues.apache.org/jira/browse/RANGER-1183
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.
> 
> 
> Diffs
> -----
> 
>   plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 
> 
> Diff: https://reviews.apache.org/r/52907/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> zhangqiang2
> 
>


Re: Review Request 52907: Change variable name

Posted by Qiang Zhang <zh...@zte.com.cn>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/
-----------------------------------------------------------

(Updated \u5341\u6708 25, 2016, 1:07 a.m.)


Review request for ranger, Don Bosco Durai, Ramesh Mani, and Velmurugan Periasamy.


Changes
-------

Review request for Ranger,Don Bosco Durai,Ramesh Mani and Velmurugan Periasamy.
thanks.


Bugs: RANGER-1183
    https://issues.apache.org/jira/browse/RANGER-1183


Repository: ranger


Description
-------

The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.


Diffs (updated)
-----

  plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 

Diff: https://reviews.apache.org/r/52907/diff/


Testing
-------


File Attachments
----------------

0001-RANGER-1183.patch
  https://reviews.apache.org/media/uploaded/files/2016/10/17/82958346-fa03-47e5-83d9-3bcc8e06e1b6__0001-RANGER-1183.patch


Thanks,

Qiang Zhang


Re: Review Request 52907: Change variable name

Posted by zh...@zte.com.cn.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/#review152840
-----------------------------------------------------------


Ship it!




Ship It!

- zhangqiang2


On \u5341\u6708 17, 2016, 6:44 a.m., zhangqiang2 wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52907/
> -----------------------------------------------------------
> 
> (Updated \u5341\u6708 17, 2016, 6:44 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-1183
>     https://issues.apache.org/jira/browse/RANGER-1183
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.
> 
> 
> Diffs
> -----
> 
>   plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 
> 
> Diff: https://reviews.apache.org/r/52907/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> 0001-RANGER-1183.patch
>   https://reviews.apache.org/media/uploaded/files/2016/10/17/82958346-fa03-47e5-83d9-3bcc8e06e1b6__0001-RANGER-1183.patch
> 
> 
> Thanks,
> 
> zhangqiang2
> 
>


Re: Review Request 52907: Change variable name

Posted by Jan Hentschel <ja...@ultratendency.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/#review152839
-----------------------------------------------------------


Ship it!




Ship It!

- Jan Hentschel


On Oct. 17, 2016, 8:44 a.m., zhangqiang2 wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52907/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2016, 8:44 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-1183
>     https://issues.apache.org/jira/browse/RANGER-1183
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.
> 
> 
> Diffs
> -----
> 
>   plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 
> 
> Diff: https://reviews.apache.org/r/52907/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> 0001-RANGER-1183.patch
>   https://reviews.apache.org/media/uploaded/files/2016/10/17/82958346-fa03-47e5-83d9-3bcc8e06e1b6__0001-RANGER-1183.patch
> 
> 
> Thanks,
> 
> zhangqiang2
> 
>


Re: Review Request 52907: Change variable name

Posted by zh...@zte.com.cn.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/
-----------------------------------------------------------

(Updated \u5341\u6708 17, 2016, 6:44 a.m.)


Review request for ranger.


Bugs: RANGER-1183
    https://issues.apache.org/jira/browse/RANGER-1183


Repository: ranger


Description
-------

The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.


Diffs
-----

  plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 

Diff: https://reviews.apache.org/r/52907/diff/


Testing
-------


File Attachments (updated)
----------------

0001-RANGER-1183.patch
  https://reviews.apache.org/media/uploaded/files/2016/10/17/82958346-fa03-47e5-83d9-3bcc8e06e1b6__0001-RANGER-1183.patch


Thanks,

zhangqiang2


Re: Review Request 52907: Change variable name

Posted by zh...@zte.com.cn.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52907/
-----------------------------------------------------------

(Updated \u5341\u6708 15, 2016, 7:22 a.m.)


Review request for ranger.


Summary (updated)
-----------------

Change variable name


Bugs: RANGER-1183
    https://issues.apache.org/jira/browse/RANGER-1183


Repository: ranger


Description
-------

The code in the file YarnClient.java has variable naming errors,the variable name is yanrQname.


Diffs
-----

  plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 9b283a2 

Diff: https://reviews.apache.org/r/52907/diff/


Testing
-------


Thanks,

zhangqiang2