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



ambari-common/src/main/python/resource_management/libraries/functions/install_jdbc_driver.py
<https://reviews.apache.org/r/30702/#comment117210>

    We should just copy the JDBC driver from the cache path. This ensures same 
version of the driver is copied over in the classpath.



ambari-server/src/main/resources/stacks/HDPWIN/2.1/hooks/after-INSTALL/scripts/hook.py
<https://reviews.apache.org/r/30702/#comment117212>

    I dont see a need to add another function for this
    
    See : 
ambari-server/src/main/resources/stacks/HDPWIN/2.1/services/OOZIE/package/scripts/oozie_server.py
    
        
force_download_file(os.path.join(params.config['hostLevelParams']['jdk_location'],
 "sqljdbc4.jar"),
          os.path.join(params.oozie_root, "extra_libs", "sqljdbc4.jar")
        )


- Jayush Luniya


On Feb. 6, 2015, 12:03 a.m., Florian Barca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30702/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2015, 12:03 a.m.)
> 
> 
> Review request for Ambari, Artem Baranchuk, Eugene Chekanskiy, Jayush Luniya, 
> and Mahadev Konar.
> 
> 
> Bugs: AMBARI-9505
>     https://issues.apache.org/jira/browse/AMBARI-9505
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Copying the SQL Server JDBC driver into the hadoop shared lib directory.
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/__init__.py
>  9b32b92cc8845d775c89f1715f4ae8d98c365843 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/install_jdbc_driver.py
>  PRE-CREATION 
>   
> ambari-server/src/main/resources/stacks/HDPWIN/2.1/hooks/after-INSTALL/scripts/hook.py
>  31df28ddb174a689717978de1cb3fce8603cd11b 
> 
> Diff: https://reviews.apache.org/r/30702/diff/
> 
> 
> Testing
> -------
> 
> mvn clean install
> Multi-node deployment on a local VM cluster -> Hive deploys and starts ok
> 
> 
> Thanks,
> 
> Florian Barca
> 
>

Reply via email to