Repository: spark Updated Branches: refs/heads/master 987eb8fad -> e92befcb4
[MINOR][DOC] modified issue link and updated status ## What changes were proposed in this pull request? This PR aims to clarify some outdated comments that i found at **spark-catalyst** and **spark-sql** pom files. Maven bug still happening and in order to track it I have updated the issue link and also the status of the issue. Author: Marcos P <[email protected]> Closes #18374 from mpenate/fix/mng-3559-comment. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/e92befcb Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/e92befcb Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/e92befcb Branch: refs/heads/master Commit: e92befcb4b57c3e4afe57b6de1622ac72e7d819c Parents: 987eb8f Author: Marcos P <[email protected]> Authored: Wed Jun 21 15:34:10 2017 +0100 Committer: Sean Owen <[email protected]> Committed: Wed Jun 21 15:34:10 2017 +0100 ---------------------------------------------------------------------- sql/catalyst/pom.xml | 2 +- sql/core/pom.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/e92befcb/sql/catalyst/pom.xml ---------------------------------------------------------------------- diff --git a/sql/catalyst/pom.xml b/sql/catalyst/pom.xml index 36948ba..0bbf7a9 100644 --- a/sql/catalyst/pom.xml +++ b/sql/catalyst/pom.xml @@ -109,7 +109,7 @@ so that the tests classes of external modules can use them. The two execution profiles are necessary - first one for 'mvn package', second one for 'mvn test-compile'. Ideally, 'mvn compile' should not compile test classes and therefore should not need this. - However, an open Maven bug (http://jira.codehaus.org/browse/MNG-3559) + However, a closed due to "Cannot Reproduce" Maven bug (https://issues.apache.org/jira/browse/MNG-3559) causes the compilation to fail if catalyst test-jar is not generated. Hence, the second execution profile for 'mvn test-compile'. --> http://git-wip-us.apache.org/repos/asf/spark/blob/e92befcb/sql/core/pom.xml ---------------------------------------------------------------------- diff --git a/sql/core/pom.xml b/sql/core/pom.xml index 7327c9b..1bc34a6 100644 --- a/sql/core/pom.xml +++ b/sql/core/pom.xml @@ -161,7 +161,7 @@ so that the tests classes of external modules can use them. The two execution profiles are necessary - first one for 'mvn package', second one for 'mvn test-compile'. Ideally, 'mvn compile' should not compile test classes and therefore should not need this. - However, an open Maven bug (http://jira.codehaus.org/browse/MNG-3559) + However, a closed due to "Cannot Reproduce" Maven bug (https://issues.apache.org/jira/browse/MNG-3559) causes the compilation to fail if catalyst test-jar is not generated. Hence, the second execution profile for 'mvn test-compile'. --> --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
