---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178861
---
Ship it!
Ship It!
- Mehul Parikh
On June 19, 2017, 11:41 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178225
---
Ship it!
Ship It!
- Colm O hEigeartaigh
On June 19, 2017, 11
> On June 19, 2017, 11:22 a.m., Colm O hEigeartaigh wrote:
> > Sorry, just one more change - I think instead of having "xForwardedURL" as
> > a class variable, instead it would be better to return it from
> > "constructForwardableURL".
Thank Colm, updated the patch.
- Ankita
--
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/
---
(Updated June 19, 2017, 11:41 a.m.)
Review request for ranger.
Changes
--
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178220
---
Sorry, just one more change - I think instead of having "xForwarde
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178219
---
Ship it!
Ship It!
- Velmurugan Periasamy
On June 19, 2017, 1
> On June 19, 2017, 10:10 a.m., Colm O hEigeartaigh wrote:
> > OK thanks. Just one other change I'd like to see - the code to construct
> > the "xForwardedURL" could all be moved into a separate method.
Thanks Colm, updated the patch with suggested change.
- Ankita
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/
---
(Updated June 19, 2017, 10:26 a.m.)
Review request for ranger.
Changes
--
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178216
---
OK thanks. Just one other change I'd like to see - the code to con
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/
---
(Updated June 19, 2017, 9:44 a.m.)
Review request for ranger.
Bugs: RANGER-16
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/#review178207
---
A few trivial code comments:
a) String xForwardedPorto -> Should
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60192/
---
Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni,
Madhan
12 matches
Mail list logo