> On Sept. 9, 2021, 1:08 p.m., Pradeep Agrawal wrote:
> > What if somebody want to use old jar due to environemnt constraints ?
> 
> Tsung-Ju Lii wrote:
>     It is probably fine, this is going to affect only the master branch, i.e. 
> the branch that's supposed to be up to date.
>     
>     What happened to me was I pulled the repo from Github, tried to run it 
> and got a lot of warnings about the driver being deprecated. The deployment 
> failed but there were way too many deprecation warnings that made locating 
> the actual error message extremely difficult.

I have tried the new jars , new jars are having backward compatibility, though 
there are deprecate warning messages installation continues. Can you post the 
details of the deployement failure.
Also please think about providing the driver details more configurable, i will 
also work on that later.


- Pradeep


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


On Sept. 1, 2021, 11:46 p.m., Tsung-Ju Lii wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73563/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2021, 11:46 p.m.)
> 
> 
> Review request for ranger.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> While deplolying Ranger with MySQL, during setup a lot of warnings for 
> `com.mysql.jdbc.Driver` being deprecated showed up. This patch changes all 
> occurrences of the said class to the new class `com.mysql.cj.jdbc.Driver`.
> 
> 
> Diffs
> -----
> 
>   jisql/src/main/java/org/apache/util/sql/Jisql.java b3e272550 
>   jisql/src/main/java/org/apache/util/sql/MySQLPLRunner.java 308350f0e 
>   migration-util/ambari2.1-hdp2.3-ranger0.50/bin/import_ranger_to_ambari.py 
> 195a116dd 
> 
> 
> Diff: https://reviews.apache.org/r/73563/diff/1/
> 
> 
> Testing
> -------
> 
> I was able to deploy successfully after I applied this patch.
> 
> 
> Thanks,
> 
> Tsung-Ju Lii
> 
>

Reply via email to