Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/314
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user kishorvpatil commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-76749927
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-67888564
Enough changed with the docs update, that I opted to just add the images
in. If we want to move some of the other markdown documents over we can, but
for now I just left
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-67103354
@revans2 can you please upmerge your patch. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-65801449
With pull #330 now up, I will wait for it to go in, and then rebase my
changes on top of it.
---
If your project is set up for it, you can reply to this email and have
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63332256
I was curious if we wanted to keep a 5MB file in our github repo that
everyone must download to get storm? As it stands now the git repo is about
11MB total. This could
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63336734
@revans2 There is another JIRA asking for the current storm.apache.org docs
to be moved onto github.
https://issues.apache.org/jira/browse/STORM-544.
As
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63337668
OK so the next question I have is, should we split the one big SVG file up
along side rasterized PNG equivalents for use in the docs?
---
If your project is set up for
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63338507
@revans2 sounds like a good idea to provide PNG files.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63354593
OK I split them apart, cleaned up one of the SVG images to make it smaller,
and added in some PNG versions, along with updating the security document to
use one of the
Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/314#issuecomment-63146118
Amazing for sharing high-res svg diagrams! Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
11 matches
Mail list logo