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




dev-support/hive-personality.sh
Lines 29 (patched)
<https://reviews.apache.org/r/59310/#comment248768>

    'trunk' should be 'master' in Apache Hive.


- Akira Ajisaka


On 5月 16, 2017, 1:12 p.m., Peter Vary wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59310/
> -----------------------------------------------------------
> 
> (Updated 5月 16, 2017, 1:12 p.m.)
> 
> 
> Review request for hive, Akira Ajisaka, Marta Kuczora, Siddharth Seth, Thejas 
> Nair, Vihang Karajgaonkar, and Barna Zsombor Klara.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Integrated Apache Yetus (https://yetus.apache.org/)
> - Created hive personality to contain the hive specific configurations
> - Created yetus-wrapper.sh to download yetus and patch the not yet released 
> changes:
> -- checkstyle_YETUS-484.sh
> -- findbugs_YETUS-471.sh
> -- maven_YETUS-506.sh
> - Created test-patch.sh, and smart-apply-patch.sh to simplify 
> testing/applying patch
> - Added patchprocess directory to gitignore
> - Two small modification in pom.xml
> -- Adding findbugs configuration to build
> -- Disabling failure on javadoc errors
> 
> 
> Diffs
> -----
> 
>   .gitignore 47c59da 
>   dev-support/checkstyle_YETUS-484.sh PRE-CREATION 
>   dev-support/findbugs_YETUS-471.sh PRE-CREATION 
>   dev-support/hive-personality.sh PRE-CREATION 
>   dev-support/maven_YETUS-506.sh PRE-CREATION 
>   dev-support/smart-apply-patch.sh PRE-CREATION 
>   dev-support/test-patch.sh PRE-CREATION 
>   dev-support/yetus-wrapper.sh PRE-CREATION 
>   pom.xml 30fa50b 
> 
> 
> Diff: https://reviews.apache.org/r/59310/diff/1/
> 
> 
> Testing
> -------
> 
> To test a patch set the FINDBUGS_HOME, and go to the hive source root 
> directory and run the test_patch.sh command like this:
> ./[petervary:~/dev/upstream/hive] $ export 
> FINDBUGS_HOME=~/dev/upstream/findbugs-3.0.1/
> ./[petervary:~/dev/upstream/hive] $ ./dev-support/test-patch.sh 
> ~/Downloads/HIVE-16345.2.patch
> 
> 
> Thanks,
> 
> Peter Vary
> 
>

Reply via email to