Skip to content

Conversation

@pwoody
Copy link

@pwoody pwoody commented Nov 14, 2016

@pwoody
Copy link
Author

pwoody commented Nov 14, 2016

This passes hive and sql tests locally

@schlosna
Copy link

@pwoody just to confirm, this is essentially apache#15835 from upstream + revert & fixing up of Duffy's changes, right?

@schlosna
Copy link

👍 LGTM

@pwoody
Copy link
Author

pwoody commented Nov 14, 2016

Yep, that is the case @schlosna . Minor tweak since we are ahead on parquet version as well

@pwoody pwoody merged commit a3b7ad5 into master Nov 14, 2016
@robert3005 robert3005 deleted the pw/parquetPruning branch November 14, 2016 19:32
pwoody pushed a commit that referenced this pull request Nov 17, 2016
ash211 pushed a commit that referenced this pull request Feb 16, 2017
* Use "extraTestArgLine" to pass extra options to scalatest.

Because the "argLine" option of scalatest is set in pom.xml and we can't
overwrite it from the command line.

Ref apache-spark-on-k8s#37

* Added a default value for extraTestArgLine

* Use a better name.

* Added a tip for this in the dev docs.
mccheah pushed a commit that referenced this pull request Apr 27, 2017
* Use "extraTestArgLine" to pass extra options to scalatest.

Because the "argLine" option of scalatest is set in pom.xml and we can't
overwrite it from the command line.

Ref apache-spark-on-k8s#37

* Added a default value for extraTestArgLine

* Use a better name.

* Added a tip for this in the dev docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants