Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Aug 1, 2015

Enforce Maven 3.3.3+ in the build. (Also update the scala compiler plugin while we're at it.)

@SparkQA
Copy link

SparkQA commented Aug 1, 2015

Test build #39350 has finished for PR 7852 at commit 3093039.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public static final class BytesToBytesMapIterator implements Iterator<Location>
    • public abstract class UnsafeKeyValueSorter
    • class SpecificUnsafeProjection extends $
    • abstract class UnsafeRowJoiner
    • |class SpecificUnsafeRowJoiner extends $
    • case class SortArray(base: Expression, ascendingOrder: Expression)
    • case class SubstringIndex(strExpr: Expression, delimExpr: Expression, countExpr: Expression)
    • public abstract class KVIterator<K, V>

@JoshRosen
Copy link
Contributor

LGTM, especially given that we've run into problems with Maven versions in the recent past; better to fail-fast here.

@asfgit asfgit closed this in 9d1c025 Aug 2, 2015
@srowen srowen deleted the SPARK-9521 branch August 2, 2015 08:44
@nchammas
Copy link
Contributor

nchammas commented Aug 3, 2015

For the record, this PR is kind of a follow up to #6770.

srowen added a commit to srowen/spark that referenced this pull request Aug 3, 2015
asfgit pushed a commit that referenced this pull request Aug 4, 2015
Follow on for #7852: Building Spark doc needs to refer to new Maven requirement too

Author: Sean Owen <[email protected]>

Closes #7905 from srowen/SPARK-9521.2 and squashes the following commits:

73285df [Sean Owen] Follow on for #7852: Building Spark doc needs to refer to new Maven requirement too
asfgit pushed a commit that referenced this pull request Aug 4, 2015
Follow on for #7852: Building Spark doc needs to refer to new Maven requirement too

Author: Sean Owen <[email protected]>

Closes #7905 from srowen/SPARK-9521.2 and squashes the following commits:

73285df [Sean Owen] Follow on for #7852: Building Spark doc needs to refer to new Maven requirement too

(cherry picked from commit 0afa6fb)
Signed-off-by: Sean Owen <[email protected]>
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.

4 participants