Re: Review Request 64230: SENTRY-2082: Exclude javax.servlet-3.0.0.v201112011016.jar from Sentry dist

2017-11-30 Thread kalyan kumar kalvagadda via Review Board


> On Dec. 1, 2017, 1:23 a.m., kalyan kumar kalvagadda wrote:
> > Ship It!

Sergio, There were test failures. may not be related but I have re-submitted 
your patch again.


- kalyan kumar


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


On Nov. 30, 2017, 11:05 p.m., Sergio Pena wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64230/
> ---
> 
> (Updated Nov. 30, 2017, 11:05 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-2082
> https://issues.apache.org/jira/browse/SENTRY-2082
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> The javax.servlet-3.0.0.v201112011016.jar transitive dependency is causing 
> some error exceptions on the Sentry 2.0 distribution in some Mac environments.
> 
> 
> Diffs
> -
> 
>   pom.xml 948f88cff41dc816f083995f795cff6b4b416f28 
> 
> 
> Diff: https://reviews.apache.org/r/64230/diff/1/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>



Re: Review Request 64230: SENTRY-2082: Exclude javax.servlet-3.0.0.v201112011016.jar from Sentry dist

2017-11-30 Thread kalyan kumar kalvagadda via Review Board

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


Ship it!




Ship It!

- kalyan kumar kalvagadda


On Nov. 30, 2017, 11:05 p.m., Sergio Pena wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64230/
> ---
> 
> (Updated Nov. 30, 2017, 11:05 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-2082
> https://issues.apache.org/jira/browse/SENTRY-2082
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> The javax.servlet-3.0.0.v201112011016.jar transitive dependency is causing 
> some error exceptions on the Sentry 2.0 distribution in some Mac environments.
> 
> 
> Diffs
> -
> 
>   pom.xml 948f88cff41dc816f083995f795cff6b4b416f28 
> 
> 
> Diff: https://reviews.apache.org/r/64230/diff/1/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>



Re: Review Request 64230: SENTRY-2082: Exclude javax.servlet-3.0.0.v201112011016.jar from Sentry dist

2017-11-30 Thread Alexander Kolbasov

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


Ship it!




Ship It!

- Alexander Kolbasov


On Nov. 30, 2017, 11:05 p.m., Sergio Pena wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64230/
> ---
> 
> (Updated Nov. 30, 2017, 11:05 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-2082
> https://issues.apache.org/jira/browse/SENTRY-2082
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> The javax.servlet-3.0.0.v201112011016.jar transitive dependency is causing 
> some error exceptions on the Sentry 2.0 distribution in some Mac environments.
> 
> 
> Diffs
> -
> 
>   pom.xml 948f88cff41dc816f083995f795cff6b4b416f28 
> 
> 
> Diff: https://reviews.apache.org/r/64230/diff/1/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>



Review Request 64230: SENTRY-2082: Exclude javax.servlet-3.0.0.v201112011016.jar from Sentry dist

2017-11-30 Thread Sergio Pena via Review Board

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

Review request for sentry.


Bugs: SENTRY-2082
https://issues.apache.org/jira/browse/SENTRY-2082


Repository: sentry


Description
---

The javax.servlet-3.0.0.v201112011016.jar transitive dependency is causing some 
error exceptions on the Sentry 2.0 distribution in some Mac environments.


Diffs
-

  pom.xml 948f88cff41dc816f083995f795cff6b4b416f28 


Diff: https://reviews.apache.org/r/64230/diff/1/


Testing
---


Thanks,

Sergio Pena