[ https://issues.apache.org/jira/browse/STORM-876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15007523#comment-15007523 ]
ASF GitHub Bot commented on STORM-876: -------------------------------------- Github user knusbaum commented on a diff in the pull request: https://github.com/apache/storm/pull/845#discussion_r44995641 --- Diff: storm-core/src/clj/backtype/storm/daemon/supervisor.clj --- @@ -326,16 +331,29 @@ (log-error t "Error when processing event") (exit-process! 20 "Error when processing an event") )) + :blob-update-timer (mk-timer :kill-fn (fn [t] + (log-error t "Error when processing blob-update") + (exit-process! 20 "Error when processing a blob-update") + )) + :localizer (Utils/createLocalizer conf (supervisor-local-dir conf)) :assignment-versions (atom {}) :sync-retry (atom 0) - :code-distributor (mk-code-distributor conf) :download-lock (Object.) :stormid->profiler-actions (atom {}) }) +(defn required-topo-files-exist? + [conf storm-id] + (let [stormroot (supervisor-stormdist-root conf storm-id) + stormjarpath (supervisor-stormjar-path stormroot) + stormcodepath (supervisor-stormcode-path stormroot) + stormconfpath (supervisor-stormconf-path stormroot)] + (and (every? exists-file? [stormroot stormconfpath stormcodepath]) + (or (local-mode? conf) + (exists-file? stormjarpath))))) --- End diff -- Indentation on 352 and 353 is wrong. Should be inline with previous args. > Dist Cache: Basic Functionality > ------------------------------- > > Key: STORM-876 > URL: https://issues.apache.org/jira/browse/STORM-876 > Project: Apache Storm > Issue Type: Improvement > Components: storm-core > Reporter: Robert Joseph Evans > Assignee: Robert Joseph Evans > Attachments: DISTCACHE.md, DistributedCacheDesignDocument.pdf > > > Basic functionality for the Dist Cache feature. > As part of this a new API should be added to support uploading and > downloading dist cache items. storm-core.ser, storm-conf.ser and storm.jar > should be written into the blob store instead of residing locally. We need a > default implementation of the blob store that does essentially what nimbus > currently does and does not need anything extra. But having an HDFS backend > too would be great for scalability and HA. > The supervisor should provide a way to download and manage these blobs and > provide a working directory for the worker process with symlinks to the > blobs. It should also allow the blobs to be updated and switch the symlink > atomically to point to the new blob once it is downloaded. > All of this is already done by code internal to Yahoo! we are in the process > of getting it ready to push back to open source shortly. -- This message was sent by Atlassian JIRA (v6.3.4#6332)