Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[STORM-2431]the default blobstore.dir is storm.local.dir/blobs which is different from distcache-blobstore.md #2033

Merged
merged 1 commit into from
Apr 5, 2017

Conversation

liu-zhaokun
Copy link
Contributor

https://issues.apache.org/jira/browse/STORM-2431?filter=-2
The default blobstore.dir is storm.local.dir/blobs which is different from distcache-blobstore.md.You can see the pictures below.The default blobstore.dir is set to storm.local.dir/blobs.

@liu-zhaokun
Copy link
Contributor Author

Hello,@HeartSaVioR
This is the new PR to modify the distcache-blobstore.md. Would you review it soon?Thanks .

@liu-zhaokun
Copy link
Contributor Author

Hello, @HeartSaVioR
this is the new PR for [STORM-2431]the default blobstore.dir is storm.local.dir/blobs which is different from distcache-blobstore.md.

@liu-zhaokun
Copy link
Contributor Author

Hello, @HeartSaVioR I am sorry to bother you.
The CI was failed,need I do anything to solve it?But I think there is nothing to cause it to fail.

@HeartSaVioR
Copy link
Contributor

No need to worry about build failure for document update. I'm stuck with other things, and also 1.1.0 vote is not announced so I would like to take a look at pending PRs after that.

@HeartSaVioR
Copy link
Contributor

+1

@liu-zhaokun
Copy link
Contributor Author

@HeartSaVioR
Need I do something before merging this PR?

@liu-zhaokun
Copy link
Contributor Author

Hi, @HeartSaVioR
Will this PR be merged?

@asfgit asfgit merged commit 488c38b into apache:master Apr 5, 2017
@HeartSaVioR
Copy link
Contributor

Sorry missed that. Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants