Re: Permissions to edit "Want to Contribute to Apache Ranger?" wiki page

2021-10-04 Thread Martin Grigorov
On Mon, Oct 4, 2021 at 3:00 AM Velmurugan Periasamy  wrote:

> Hi Martin - I have added you to the edit permissions. Can you please try
> now? Thanks for your help.
>

Thank you, Velmurugan!
I've made the changes! I hope it is more clear now for future contributors!

Regards,
Martin


>
>
> On Fri, Oct 1, 2021 at 3:25 AM Martin Tzvetanov Grigorov <
> mgrigo...@apache.org> wrote:
>
>
> On 2021/09/28 08:12:06, Martin Grigorov  mgrigo...@apache.org>> wrote:
> > Hello Ranger devs,
> >
> > Could you please give me permissions to edit
> >
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=55151244
> <https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=55151244>
> ?
> > I'd like to fix the url to GitBox (ex. git-wip-us) and add few more
> details
> > about the Pull Requests at GitHub.
> >
> > My apache id is: mgrigorov
> >
> > Thank you!
>
> Ping!
>
> >
> > Regards,
> > Martin
> >


Permissions to edit "Want to Contribute to Apache Ranger?" wiki page

2021-09-28 Thread Martin Grigorov
Hello Ranger devs,

Could you please give me permissions to edit
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=55151244 ?
I'd like to fix the url to GitBox (ex. git-wip-us) and add few more details
about the Pull Requests at GitHub.

My apache id is: mgrigorov

Thank you!

Regards,
Martin


Re: Review Request 73283: Use OpenJDK instead of OracleJDK at TravisCI

2021-09-27 Thread Martin Grigorov

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

(Updated Sept. 27, 2021, 6:59 a.m.)


Review request for ranger.


Repository: ranger


Description
---

Test JDK 8 & 11 because they are Long Term supported


Diffs (updated)
-

  .travis.yml deac951610 
  security-admin/pom.xml eac012049f 
  unixauthpam/pom.xml 4d5b05e755 


Diff: https://reviews.apache.org/r/73283/diff/4/

Changes: https://reviews.apache.org/r/73283/diff/3-4/


Testing
---


Thanks,

Martin Grigorov



Re: Review Request 73281: Fix compilation problems for Java 8 and Java 11.

2021-09-23 Thread Martin Grigorov


> On Sept. 23, 2021, 10:11 a.m., Dhaval Shah wrote:
> > Build succeeded with patch on java 8 and java 11.

Thanks for the review, Dhaval Shah!


- Martin


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


On Sept. 21, 2021, 7:18 a.m., Martin Grigorov wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73281/
> ---
> 
> (Updated Sept. 21, 2021, 7:18 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Fix build due to hive exec and JDK 8 specific class
> 
> 
> Diffs
> -
> 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 
> 371e367c1 
>   pom.xml 36036a6ad 
>   ugsync-util/pom.xml aa992cabf 
>   ugsync/pom.xml 07e0c5872 
>   
> ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java
>  30820ce33 
> 
> 
> Diff: https://reviews.apache.org/r/73281/diff/4/
> 
> 
> Testing
> ---
> 
> Issue 1) 
> org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory
>  was not able to import com.google.common.base.Throwables.throwIfUnchecked 
> because it was coming from an old copy of Guava's Throwables shaded in 
> hive-exec.
> By replacing hive-exec with orc-core in agents-audit module all depending 
> modules use their preferred version of Guava (26.0-jre)
> 
> Issue 2) RangerSafenetKeySecure.java uses directly 
> sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and 
> this breaks the compilation.
> This should use reflection to load SunPKCS11 class dynamically.
> 
> Issue 3) JAXB is no more part of JDK 11+
> 
> Issue 4) JUnit 4 Assert.assertThat() is deprecated in favour of Hamcrest's 
> MatcherAssert.assertThat(). This fixes a compilation issue with JDK11.
> 
> 
> Thanks,
> 
> Martin Grigorov
> 
>



Re: Review Request 73283: Use OpenJDK instead of OracleJDK at TravisCI

2021-09-21 Thread Martin Grigorov

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

(Updated Sept. 21, 2021, 8:08 a.m.)


Review request for ranger.


Repository: ranger


Description
---

Test JDK 8 & 11 because they are Long Term supported


Diffs (updated)
-

  .travis.yml deac95161 


Diff: https://reviews.apache.org/r/73283/diff/3/

Changes: https://reviews.apache.org/r/73283/diff/2-3/


Testing
---


Thanks,

Martin Grigorov



Re: Review Request 73282: Update test dependencies and Maven plugins

2021-09-21 Thread Martin Grigorov

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

(Updated Sept. 21, 2021, 7:11 a.m.)


Review request for ranger.


Repository: ranger


Description
---

Several test related dependencies and Maven plugins could be easily updated.


Diffs (updated)
-

  pom.xml 36036a6ad 
  security-admin/pom.xml eac012049 
  ugsync/pom.xml 07e0c5872 


Diff: https://reviews.apache.org/r/73282/diff/2/

Changes: https://reviews.apache.org/r/73282/diff/1-2/


Testing
---


Thanks,

Martin Grigorov



Re: Review Request 73281: Fix compilation problems for Java 8 and Java 11.

2021-09-21 Thread Martin Grigorov

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

(Updated Sept. 21, 2021, 7:18 a.m.)


Review request for ranger.


Repository: ranger


Description
---

Fix build due to hive exec and JDK 8 specific class


Diffs (updated)
-

  kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 
371e367c1 
  pom.xml 36036a6ad 
  ugsync-util/pom.xml aa992cabf 
  ugsync/pom.xml 07e0c5872 
  
ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java
 30820ce33 


Diff: https://reviews.apache.org/r/73281/diff/4/

Changes: https://reviews.apache.org/r/73281/diff/3-4/


Testing
---

Issue 1) 
org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory
 was not able to import com.google.common.base.Throwables.throwIfUnchecked 
because it was coming from an old copy of Guava's Throwables shaded in 
hive-exec.
By replacing hive-exec with orc-core in agents-audit module all depending 
modules use their preferred version of Guava (26.0-jre)

Issue 2) RangerSafenetKeySecure.java uses directly 
sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and 
this breaks the compilation.
This should use reflection to load SunPKCS11 class dynamically.

Issue 3) JAXB is no more part of JDK 11+

Issue 4) JUnit 4 Assert.assertThat() is deprecated in favour of Hamcrest's 
MatcherAssert.assertThat(). This fixes a compilation issue with JDK11.


Thanks,

Martin Grigorov



Re: Review Request 73281: Fix compilation problems for Java 8 and Java 11.

2021-04-29 Thread Martin Grigorov

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

(Updated April 29, 2021, 7:43 a.m.)


Review request for ranger.


Changes
---

Update to latest master where issue 1) is fixed.


Repository: ranger


Description
---

Fix build due to hive exec and JDK 8 specific class


Diffs (updated)
-

  kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 
371e367c1 
  pom.xml e508dd1f8 
  ugsync-util/pom.xml aa992cabf 
  ugsync/pom.xml 07e0c5872 
  
ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java
 30820ce33 


Diff: https://reviews.apache.org/r/73281/diff/3/

Changes: https://reviews.apache.org/r/73281/diff/2-3/


Testing
---

Issue 1) 
org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory
 was not able to import com.google.common.base.Throwables.throwIfUnchecked 
because it was coming from an old copy of Guava's Throwables shaded in 
hive-exec.
By replacing hive-exec with orc-core in agents-audit module all depending 
modules use their preferred version of Guava (26.0-jre)

Issue 2) RangerSafenetKeySecure.java uses directly 
sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and 
this breaks the compilation.
This should use reflection to load SunPKCS11 class dynamically.

Issue 3) JAXB is no more part of JDK 11+

Issue 4) JUnit 4 Assert.assertThat() is deprecated in favour of Hamcrest's 
MatcherAssert.assertThat(). This fixes a compilation issue with JDK11.


Thanks,

Martin Grigorov



Re: Review Request 73283: Use OpenJDK instead of OracleJDK at TravisCI

2021-04-19 Thread Martin Grigorov

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

(Updated April 19, 2021, 12:13 p.m.)


Review request for ranger.


Changes
---

Install OpenJDK from Ubuntu repositories because the one provided by TravisCI 
is too old, e.g. 11.0.2 instead of 11.0.10


Repository: ranger


Description
---

Test JDK 8 & 11 because they are Long Term supported


Diffs (updated)
-

  .travis.yml deac95161 


Diff: https://reviews.apache.org/r/73283/diff/2/

Changes: https://reviews.apache.org/r/73283/diff/1-2/


Testing
---


Thanks,

Martin Grigorov



Review Request 73283: Use OpenJDK instead of OracleJDK at TravisCI

2021-04-16 Thread Martin Grigorov

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

Review request for ranger.


Repository: ranger


Description
---

Test JDK 8 & 11 because they are Long Term supported


Diffs
-

  .travis.yml deac95161 


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


Testing
---


Thanks,

Martin Grigorov



Review Request 73282: Update test dependencies and Maven plugins

2021-04-16 Thread Martin Grigorov

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

Review request for ranger.


Repository: ranger


Description
---

Several test related dependencies and Maven plugins could be easily updated.


Diffs
-

  pom.xml fe7812bd7 
  security-admin/pom.xml 02edf9143 
  ugsync/pom.xml 07e0c5872 


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


Testing
---


Thanks,

Martin Grigorov



Re: Review Request 73281: Fix compilation problems for Java 8 and Java 11.

2021-04-16 Thread Martin Grigorov

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

(Updated April 16, 2021, 8:08 a.m.)


Review request for ranger.


Repository: ranger


Description
---

Fix build due to hive exec and JDK 8 specific class


Diffs (updated)
-

  agents-audit/pom.xml 06ed81f55 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 
371e367c1 
  pom.xml fe7812bd7 
  ugsync-util/pom.xml aa992cabf 
  ugsync/pom.xml 07e0c5872 
  
ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java
 30820ce33 


Diff: https://reviews.apache.org/r/73281/diff/2/

Changes: https://reviews.apache.org/r/73281/diff/1-2/


Testing (updated)
---

Issue 1) 
org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory
 was not able to import com.google.common.base.Throwables.throwIfUnchecked 
because it was coming from an old copy of Guava's Throwables shaded in 
hive-exec.
By replacing hive-exec with orc-core in agents-audit module all depending 
modules use their preferred version of Guava (26.0-jre)

Issue 2) RangerSafenetKeySecure.java uses directly 
sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and 
this breaks the compilation.
This should use reflection to load SunPKCS11 class dynamically.

Issue 3) JAXB is no more part of JDK 11+

Issue 4) JUnit 4 Assert.assertThat() is deprecated in favour of Hamcrest's 
MatcherAssert.assertThat(). This fixes a compilation issue with JDK11.


Thanks,

Martin Grigorov



Review Request 73281: Fix compilation problems for Java 8 and Java 11.

2021-04-15 Thread Martin Grigorov

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

Review request for ranger.


Repository: ranger


Description
---

Fix build due to hive exec and JDK 8 specific class


Diffs
-

  agents-audit/pom.xml 06ed81f55c 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 
371e367c18 
  pom.xml fe7812bd72 


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


Testing
---

Issue 1) 
org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory
 was not able to import com.google.common.base.Throwables.throwIfUnchecked 
because it was coming from an old copy of Guava's Throwables shaded in 
hive-exec.
By replacing hive-exec with orc-core in agents-audit module all depending 
modules use their preferred version of Guava (26.0-jre)

Issue 2) RangerSafenetKeySecure.java uses directly 
sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and 
this breaks the compilation.
This should use reflection to load SunPKCS11 class dynamically.


Thanks,

Martin Grigorov