[
https://issues.apache.org/jira/browse/STORM-239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14059304#comment-14059304
]
ASF GitHub Bot commented on STORM-239:
--------------------------------------
Github user apetresc commented on the pull request:
https://github.com/apache/incubator-storm/pull/188#issuecomment-48779351
Hey @d2r,
Thanks, I've made your recommended changes (although there was no `else`
clause in your `if-let`, and I'm not sure what we could possibly do to recover
if `(.getResource classloader RESOURCES-SUBDIR)` was `nil` anyway, so I just
kept the `let`).
Commit amended!
> Supervisor fails in working directories whose paths have spaces in them
> -----------------------------------------------------------------------
>
> Key: STORM-239
> URL: https://issues.apache.org/jira/browse/STORM-239
> Project: Apache Storm (Incubating)
> Issue Type: Bug
> Environment: Linux
> Reporter: Adrian Petrescu
> Priority: Minor
> Labels: build
>
> This is incredibly minor, but when running the unit tests from a path that
> has a space somewhere, you get a failure of the form:
> {code}
> 7378 [Thread-5] INFO backtype.storm.daemon.supervisor - Copying resources at
> file:/var/lib/jenkins/jobs/Apache%20Storm/workspace/storm-core/target/test-classes/resources
> to
> /tmp/265ce1e8-8a68-475a-9d59-ccb599add284/supervisor/stormdist/test-1-0/resources
> 7383 [Thread-5] ERROR backtype.storm.event - Error when processing event
> java.io.FileNotFoundException: Source
> '/var/lib/jenkins/jobs/Apache%20Storm/workspace/storm-core/target/test-classes/resources'
> does not exist
> at org.apache.commons.io.FileUtils.copyDirectory(FileUtils.java:866)
> ~[commons-io-1.4.jar:1.4]
> at org.apache.commons.io.FileUtils.copyDirectory(FileUtils.java:770)
> ~[commons-io-1.4.jar:1.4]
> at org.apache.commons.io.FileUtils.copyDirectory(FileUtils.java:745)
> ~[commons-io-1.4.jar:1.4]
> at backtype.storm.daemon.supervisor$fn__4995.invoke(supervisor.clj:493)
> ~[classes/:na]
> at clojure.lang.MultiFn.invoke(MultiFn.java:172) ~[clojure-1.4.0.jar:na]
> at
> backtype.storm.daemon.supervisor$mk_synchronize_supervisor$this__4902.invoke(supervisor.clj:325)
> ~[classes/:na]
> at backtype.storm.event$event_manager$fn__2530.invoke(event.clj:39)
> ~[classes/:na]
> at clojure.lang.AFn.run(AFn.java:24) ~[clojure-1.4.0.jar:na]
> at java.lang.Thread.run(Thread.java:724) ~[na:1.7.0_25]
> 7405 [Thread-5] INFO backtype.storm.util - Halting process: ("Error when
> processing an event")
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Storm ............................................. SUCCESS [22.941s]
> [INFO] maven-shade-clojure-transformer ................... SUCCESS [23.513s]
> [INFO] Storm Core ........................................ FAILURE [2:54.488s]
> [INFO]
> ------------------------------------------------------------------------
> [INFO] BUILD FAILURE
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Total time: 3:52.857s
> [INFO] Finished at: Mon Feb 10 16:16:28 EST 2014
> [INFO] Final Memory: 29M/262M
> [INFO]
> ------------------------------------------------------------------------
> {code}
> This can be kind of annoying when (as in my example) a continuous integration
> system is running tests, since you often don't have full control of the path
> it chooses to check the source tree out at.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
