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


Ship it!




Ship It!

- Rohini Palaniswamy


On Jan. 27, 2020, 10:53 a.m., Adam Szita wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72048/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2020, 10:53 a.m.)
> 
> 
> Review request for pig, Daniel Dai, Koji Noguchi, Nandor Kollar, and Rohini 
> Palaniswamy.
> 
> 
> Bugs: PIG-4764
>     https://issues.apache.org/jira/browse/PIG-4764
> 
> 
> Repository: pig
> 
> 
> Description
> -------
> 
> Make Pig work with Hive 3.0
> 
> 
> Diffs
> -----
> 
>   trunk/build.xml 1872871 
>   trunk/ivy.xml 1872870 
>   trunk/ivy/libraries.properties 1872870 
>   
> trunk/shims/src/hadoop2/org/apache/pig/backend/hadoop/executionengine/shims/HadoopShims.java
>  1872870 
>   trunk/shims/src/hive1/org/apache/pig/hive/HiveShims.java PRE-CREATION 
>   trunk/shims/src/hive3/org/apache/pig/hive/HiveShims.java PRE-CREATION 
>   trunk/src/org/apache/pig/Expression.java 1872870 
>   trunk/src/org/apache/pig/builtin/HiveUDAF.java 1872870 
>   trunk/src/org/apache/pig/builtin/HiveUDFBase.java 1872870 
>   trunk/src/org/apache/pig/builtin/OrcStorage.java 1872870 
>   trunk/src/org/apache/pig/impl/util/hive/HiveUtils.java 1872870 
>   trunk/src/org/apache/pig/newplan/FilterExtractor.java 1872870 
>   trunk/test/org/apache/pig/builtin/TestOrcStorage.java 1872870 
>   trunk/test/org/apache/pig/builtin/TestOrcStoragePushdown.java 1872870 
>   trunk/test/org/apache/pig/test/TestLoaderStorerShipCacheFiles.java 1872870 
> 
> 
> Diff: https://reviews.apache.org/r/72048/diff/1/
> 
> 
> Testing
> -------
> 
> Ran all unit tests with both -Dhiveversion=1 and -Dhiveversion=3.
> Checked a few HiveUDFs working in grunt.
> 
> 
> Thanks,
> 
> Adam Szita
> 
>

Reply via email to