[
https://issues.apache.org/jira/browse/CALCITE-1064?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15185309#comment-15185309
]
Josh Elser commented on CALCITE-1064:
-------------------------------------
Typically, there is a dedicated "assemble" Maven module included in projects,
which depends on all other modules for some project, whose express purpose is
generating a tarball. I believe the problem here in Calcite is that the parent
pom for the entire project serves this purpose which must run before any of the
children modules is run.
Anywho, I'm messing around with this to see if I can deal with it in Avatica.
Will report back with a concrete change -- that should help clarify things.
> Unnecessary maven-remote-resources-plugin configuration
> -------------------------------------------------------
>
> Key: CALCITE-1064
> URL: https://issues.apache.org/jira/browse/CALCITE-1064
> Project: Calcite
> Issue Type: Bug
> Components: build
> Reporter: Josh Elser
> Assignee: Josh Elser
> Priority: Minor
> Fix For: 1.7.0
>
>
> Just got pinged with a question from someone about something I also noticed
> during 1.6.0-rc1 testing:
> When building from a clean checkout (or deleting the top-level target/
> directory), you'll see the calcite build trying to pull down the jars we're
> trying to build:
> {noformat}
> mvn package
> [INFO] Scanning for projects...
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Reactor Build Order:
> [INFO]
> [INFO] Calcite
> [INFO] Calcite Avatica
> [INFO] Calcite Avatica Server
> [INFO] Calcite Linq4j
> [INFO] Calcite Core
> [INFO] Calcite Examples
> [INFO] Calcite Example CSV
> [INFO] Calcite Example Function
> [INFO] Calcite MongoDB
> [INFO] Calcite Piglet
> [INFO] Calcite Plus
> [INFO] Calcite Spark
> [INFO] Calcite Splunk
> [INFO] Calcite Ubenchmark
> [INFO]
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Building Calcite 1.7.0-SNAPSHOT
> [INFO]
> ------------------------------------------------------------------------
> [INFO]
> [INFO] --- maven-checkstyle-plugin:2.12.1:check (validate) @ calcite ---
> [INFO]
> [INFO] --- git-commit-id-plugin:2.1.9:revision (default) @ calcite ---
> [INFO]
> [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ calcite ---
> [INFO]
> [INFO] --- maven-remote-resources-plugin:1.5:process (root-resources) @
> calcite ---
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/calcite-avatica-1.7.0-SNAPSHOT.jar
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/calcite-avatica-1.7.0-SNAPSHOT.jar
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-avatica/1.7.0-SNAPSHOT/calcite-avatica-1.7.0-SNAPSHOT.jar
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/calcite-linq4j-1.7.0-SNAPSHOT.jar
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/calcite-linq4j-1.7.0-SNAPSHOT.jar
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-linq4j/1.7.0-SNAPSHOT/calcite-linq4j-1.7.0-SNAPSHOT.jar
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/maven-metadata.xml
> Downloading:
> https://repository.apache.org/content/repositories/snapshots/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/calcite-core-1.7.0-SNAPSHOT.jar
> Downloading:
> http://conjars.org/repo/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/calcite-core-1.7.0-SNAPSHOT.jar
> Downloading:
> http://repository.apache.org/snapshots/org/apache/calcite/calcite-core/1.7.0-SNAPSHOT/calcite-core-1.7.0-SNAPSHOT.jar
> {noformat}
> It looks like this was introduced in CALCITE-741. The intent of the change
> isn't necesarily wrong, but I believe this is just trying to work against
> Maven.
> I think the change to make for Maven to work as it wants is to make a new
> module dedicated to creating the assembly. That module will depend on the
> other modules which should properly create the transitive dependency list.
> It's also worth noting that the maven-remote-resources-plugin configuration
> littered everywhere is duplicative (this is already handled in the Apache
> parent pom).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
