Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented May 4, 2015

Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".
@srowen
Copy link
Member

srowen commented May 4, 2015

Yes, seems prudent to manage the version for the whole project if one part is going to depend on a particular version. LGTM.

@pwendell
Copy link
Contributor

pwendell commented May 4, 2015

LGTM

@SparkQA
Copy link

SparkQA commented May 4, 2015

Test build #31778 has finished for PR 5893 at commit ea2112d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • abstract class Evaluator extends Params
    • abstract class PipelineStage extends Params with Logging
    • class BinaryClassificationEvaluator extends Evaluator with HasRawPredictionCol with HasLabelCol

asfgit pushed a commit that referenced this pull request May 5, 2015
Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Author: Marcelo Vanzin <[email protected]>

Closes #5893 from vanzin/ivy-dep-fix and squashes the following commits:

ea2112d [Marcelo Vanzin] [minor] [build] Declare ivy dependency in root pom.

(cherry picked from commit c5790a2)
Signed-off-by: Sean Owen <[email protected]>
asfgit pushed a commit that referenced this pull request May 5, 2015
Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Author: Marcelo Vanzin <[email protected]>

Closes #5893 from vanzin/ivy-dep-fix and squashes the following commits:

ea2112d [Marcelo Vanzin] [minor] [build] Declare ivy dependency in root pom.

(cherry picked from commit c5790a2)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in c5790a2 May 5, 2015
@vanzin vanzin deleted the ivy-dep-fix branch May 5, 2015 21:33
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Author: Marcelo Vanzin <[email protected]>

Closes apache#5893 from vanzin/ivy-dep-fix and squashes the following commits:

ea2112d [Marcelo Vanzin] [minor] [build] Declare ivy dependency in root pom.
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Author: Marcelo Vanzin <[email protected]>

Closes apache#5893 from vanzin/ivy-dep-fix and squashes the following commits:

ea2112d [Marcelo Vanzin] [minor] [build] Declare ivy dependency in root pom.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
Without this, any dependency that pulls ivy transitively may override
the version and potentially cause issue. In my machine, the hive tests
were pulling an old version of ivy, and subsequently failing with a
"NoSuchMethodError".

Author: Marcelo Vanzin <[email protected]>

Closes apache#5893 from vanzin/ivy-dep-fix and squashes the following commits:

ea2112d [Marcelo Vanzin] [minor] [build] Declare ivy dependency in root pom.
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