You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2017/02/15 13:55:30 UTC
arrow git commit: ARROW-561:[JAVA][PYTHON] Update java & python
dependencies to improve downstream packaging experience
Repository: arrow
Updated Branches:
refs/heads/master d50f1525a -> fa8d27f31
ARROW-561:[JAVA][PYTHON] Update java & python dependencies to improve downstream packaging experience
The current build for arrow uses a interesting work around for hamcrest conflict between JUNIT and mockito which results in mockito being in the compile scope. This is not suitable for some downstream users.
Python setup file also leaves out test dependency (not overly important but useful for developers) & we can clarify parquet-cpp as an "extra" dependency for people requiring parquet support (already mentioned in the README file but good to have clarity in setup.py as well).
Author: Holden Karau <ho...@us.ibm.com>
Closes #342 from holdenk/ARROW-561-improve-deps and squashes the following commits:
5919b32 [Holden Karau] Drop extras_requires
938ed97 [Holden Karau] Mention test requires pytest and add an extra requires for parquet
283d6cd [Holden Karau] Remove mockito from compile scope and fix hamcrest issue with exclusion rule instead
Project: http://git-wip-us.apache.org/repos/asf/arrow/repo
Commit: http://git-wip-us.apache.org/repos/asf/arrow/commit/fa8d27f3
Tree: http://git-wip-us.apache.org/repos/asf/arrow/tree/fa8d27f3
Diff: http://git-wip-us.apache.org/repos/asf/arrow/diff/fa8d27f3
Branch: refs/heads/master
Commit: fa8d27f314b7c21c611d1c5caaa9b32ae0cb2b06
Parents: d50f152
Author: Holden Karau <ho...@us.ibm.com>
Authored: Wed Feb 15 08:55:24 2017 -0500
Committer: Wes McKinney <we...@twosigma.com>
Committed: Wed Feb 15 08:55:24 2017 -0500
----------------------------------------------------------------------
java/pom.xml | 10 ++++++++--
python/setup.py | 1 +
2 files changed, 9 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/arrow/blob/fa8d27f3/java/pom.xml
----------------------------------------------------------------------
diff --git a/java/pom.xml b/java/pom.xml
index a147d66..e467cc1 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -442,11 +442,17 @@
<scope>test</scope>
</dependency>
<dependency>
- <!-- Mockito needs to be on the class path after JUnit (or Hamcrest) as
- long as Mockito _contains_ older Hamcrest classes. See arrow-2130. -->
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
<version>1.9.5</version>
+ <scope>test</scope>
+ <!-- Mockito contains old Hamcrest classes, causes issues with JUNIT -->
+ <exclusions>
+ <exclusion>
+ <groupId>org.hamcrest</groupId>
+ <artifactId>hamcrest-core</artifactId>
+ </exclusion>
+ </exclusions>
</dependency>
<dependency>
<groupId>ch.qos.logback</groupId>
http://git-wip-us.apache.org/repos/asf/arrow/blob/fa8d27f3/python/setup.py
----------------------------------------------------------------------
diff --git a/python/setup.py b/python/setup.py
index 5f5e5f3..54d1cd3 100644
--- a/python/setup.py
+++ b/python/setup.py
@@ -298,6 +298,7 @@ setup(
use_scm_version = {"root": "..", "relative_to": __file__},
setup_requires=['setuptools_scm'],
install_requires=['cython >= 0.23', 'numpy >= 1.9', 'six >= 1.0.0'],
+ test_requires=['pytest'],
description="Python library for Apache Arrow",
long_description=long_description,
classifiers=[