You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by Willem Jiang <wi...@gmail.com> on 2014/09/12 10:01:35 UTC
Re: git commit: camel-spark-rest: fixing timing issue in spark
tests. -DskipTests really skips the tests now
Hi Grzegorz,
I tried to run "mvn clean install" with JDK17 and JDK18, all the tests ran even with JDK17.
--
Willem Jiang
Red Hat, Inc.
Web: http://www.redhat.com
Blog: http://willemjiang.blogspot.com (English)
http://jnn.iteye.com (Chinese)
Twitter: willemjiang
Weibo: 姜宁willem
On September 12, 2014 at 3:27:08 PM, ggrzybek@apache.org (ggrzybek@apache.org) wrote:
> Repository: camel
> Updated Branches:
> refs/heads/master 45f12263e -> 47d169ae7
>
>
> camel-spark-rest: fixing timing issue in spark tests. -DskipTests really skips the
> tests now
>
>
> Project: http://git-wip-us.apache.org/repos/asf/camel/repo
> Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/47d169ae
> Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/47d169ae
> Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/47d169ae
>
> Branch: refs/heads/master
> Commit: 47d169ae7d1c6b5cc2192b9603b2c4f47140df48
> Parents: 45f1226
> Author: Grzegorz Grzybek
> Authored: Fri Sep 12 09:26:31 2014 +0200
> Committer: Grzegorz Grzybek
> Committed: Fri Sep 12 09:26:31 2014 +0200
>
> ----------------------------------------------------------------------
> components/camel-spark-rest/pom.xml | 16 ++--------------
> .../camel/component/sparkrest/BaseSparkTest.java | 1 +
> 2 files changed, 3 insertions(+), 14 deletions(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/pom.xml
> ----------------------------------------------------------------------
> diff --git a/components/camel-spark-rest/pom.xml b/components/camel-spark-rest/pom.xml
> index a3b18b1..c064f18 100644
> --- a/components/camel-spark-rest/pom.xml
> +++ b/components/camel-spark-rest/pom.xml
> @@ -119,15 +119,14 @@
>
> jdk8-test
>
> - 1.8
> + !1.8
>
>
>
>
> maven-surefire-plugin
>
> - false
> - 300
> + true
>
>
>
> @@ -135,15 +134,4 @@
>
>
>
> -
> -
> -
> - maven-surefire-plugin
> -
> - true
> -
> -
> -
> -
> -
>
>
> http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> ----------------------------------------------------------------------
> diff --git a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> index 77d862e..a810ace 100644
> --- a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> +++ b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> @@ -32,6 +32,7 @@ public abstract class BaseSparkTest extends CamelTestSupport {
> public void setUp() throws Exception {
> port = AvailablePortFinder.getNextAvailable(25500);
> super.setUp();
> + Thread.sleep(200);
> }
>
> @Override
>
>
Re: git commit: camel-spark-rest: fixing timing issue in spark
tests. -DskipTests really skips the tests now
Posted by Willem Jiang <wi...@gmail.com>.
Sorry, it's my fault. Now the pom works as expect after I setup the JAVA_HOME and PATH rightly.
--
Willem Jiang
Red Hat, Inc.
Web: http://www.redhat.com
Blog: http://willemjiang.blogspot.com (English)
http://jnn.iteye.com (Chinese)
Twitter: willemjiang
Weibo: 姜宁willem
On September 12, 2014 at 4:01:46 PM, Willem Jiang (willem.jiang@gmail.com) wrote:
> Hi Grzegorz,
>
> I tried to run "mvn clean install" with JDK17 and JDK18, all the tests ran even with JDK17.
>
>
> --
> Willem Jiang
>
> Red Hat, Inc.
> Web: http://www.redhat.com
> Blog: http://willemjiang.blogspot.com (English)
> http://jnn.iteye.com (Chinese)
> Twitter: willemjiang
> Weibo: 姜宁willem
>
>
>
> On September 12, 2014 at 3:27:08 PM, ggrzybek@apache.org (ggrzybek@apache.org) wrote:
> > Repository: camel
> > Updated Branches:
> > refs/heads/master 45f12263e -> 47d169ae7
> >
> >
> > camel-spark-rest: fixing timing issue in spark tests. -DskipTests really skips the
> > tests now
> >
> >
> > Project: http://git-wip-us.apache.org/repos/asf/camel/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/47d169ae
> > Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/47d169ae
> > Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/47d169ae
> >
> > Branch: refs/heads/master
> > Commit: 47d169ae7d1c6b5cc2192b9603b2c4f47140df48
> > Parents: 45f1226
> > Author: Grzegorz Grzybek
> > Authored: Fri Sep 12 09:26:31 2014 +0200
> > Committer: Grzegorz Grzybek
> > Committed: Fri Sep 12 09:26:31 2014 +0200
> >
> > ----------------------------------------------------------------------
> > components/camel-spark-rest/pom.xml | 16 ++--------------
> > .../camel/component/sparkrest/BaseSparkTest.java | 1 +
> > 2 files changed, 3 insertions(+), 14 deletions(-)
> > ----------------------------------------------------------------------
> >
> >
> > http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/components/camel-spark-rest/pom.xml b/components/camel-spark-rest/pom.xml
> > index a3b18b1..c064f18 100644
> > --- a/components/camel-spark-rest/pom.xml
> > +++ b/components/camel-spark-rest/pom.xml
> > @@ -119,15 +119,14 @@
> >
> > jdk8-test
> >
> > - 1.8
> > + !1.8
> >
> >
> >
> >
> > maven-surefire-plugin
> >
> > - false
> > - 300
> > + true
> >
> >
> >
> > @@ -135,15 +134,4 @@
> >
> >
> >
> > -
> > -
> > -
> > - maven-surefire-plugin
> > -
> > - true
> > -
> > -
> > -
> > -
> > -
> >
> >
> > http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > ----------------------------------------------------------------------
> > diff --git a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > index 77d862e..a810ace 100644
> > --- a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > +++ b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > @@ -32,6 +32,7 @@ public abstract class BaseSparkTest extends CamelTestSupport {
> > public void setUp() throws Exception {
> > port = AvailablePortFinder.getNextAvailable(25500);
> > super.setUp();
> > + Thread.sleep(200);
> > }
> >
> > @Override
> >
> >
>
>
Re: git commit: camel-spark-rest: fixing timing issue in spark tests.
-DskipTests really skips the tests now
Posted by Grzegorz Grzybek <gr...@gmail.com>.
Hi
for me camel-spark-rest was running the tests in any case (even with
-Dmaven.test.skip=true or -DskipTests or -Pfastinstall).
regards
Grzegorz
2014-09-12 10:01 GMT+02:00 Willem Jiang <wi...@gmail.com>:
> Hi Grzegorz,
>
> I tried to run "mvn clean install" with JDK17 and JDK18, all the tests ran
> even with JDK17.
>
>
> --
> Willem Jiang
>
> Red Hat, Inc.
> Web: http://www.redhat.com
> Blog: http://willemjiang.blogspot.com (English)
> http://jnn.iteye.com (Chinese)
> Twitter: willemjiang
> Weibo: 姜宁willem
>
>
>
> On September 12, 2014 at 3:27:08 PM, ggrzybek@apache.org (
> ggrzybek@apache.org) wrote:
> > Repository: camel
> > Updated Branches:
> > refs/heads/master 45f12263e -> 47d169ae7
> >
> >
> > camel-spark-rest: fixing timing issue in spark tests. -DskipTests really
> skips the
> > tests now
> >
> >
> > Project: http://git-wip-us.apache.org/repos/asf/camel/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/47d169ae
> > Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/47d169ae
> > Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/47d169ae
> >
> > Branch: refs/heads/master
> > Commit: 47d169ae7d1c6b5cc2192b9603b2c4f47140df48
> > Parents: 45f1226
> > Author: Grzegorz Grzybek
> > Authored: Fri Sep 12 09:26:31 2014 +0200
> > Committer: Grzegorz Grzybek
> > Committed: Fri Sep 12 09:26:31 2014 +0200
> >
> > ----------------------------------------------------------------------
> > components/camel-spark-rest/pom.xml | 16 ++--------------
> > .../camel/component/sparkrest/BaseSparkTest.java | 1 +
> > 2 files changed, 3 insertions(+), 14 deletions(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/components/camel-spark-rest/pom.xml
> b/components/camel-spark-rest/pom.xml
> > index a3b18b1..c064f18 100644
> > --- a/components/camel-spark-rest/pom.xml
> > +++ b/components/camel-spark-rest/pom.xml
> > @@ -119,15 +119,14 @@
> >
> > jdk8-test
> >
> > - 1.8
> > + !1.8
> >
> >
> >
> >
> > maven-surefire-plugin
> >
> > - false
> > - 300
> > + true
> >
> >
> >
> > @@ -135,15 +134,4 @@
> >
> >
> >
> > -
> > -
> > -
> > - maven-surefire-plugin
> > -
> > - true
> > -
> > -
> > -
> > -
> > -
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/camel/blob/47d169ae/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > ----------------------------------------------------------------------
> > diff --git
> a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> >
> b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > index 77d862e..a810ace 100644
> > ---
> a/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > +++
> b/components/camel-spark-rest/src/test/java/org/apache/camel/component/sparkrest/BaseSparkTest.java
> > @@ -32,6 +32,7 @@ public abstract class BaseSparkTest extends
> CamelTestSupport {
> > public void setUp() throws Exception {
> > port = AvailablePortFinder.getNextAvailable(25500);
> > super.setUp();
> > + Thread.sleep(200);
> > }
> >
> > @Override
> >
> >
>
>