brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ahgittin <...@git.apache.org>
Subject [GitHub] incubator-brooklyn pull request: Persist feeds
Date Thu, 25 Sep 2014 21:59:29 GMT
Github user ahgittin commented on the pull request:

    https://github.com/apache/incubator-brooklyn/pull/108#issuecomment-56890567
  
    @aledsage i'll either do a new PR or a merge if it looks safe.  based on #177 it looks
like i can merge master into a PR branch then merge into master and the history comes out
okay.  am trying the two-step merge here (where i merged with flaws, so that my fixes are
clearly in a new commit).
    
    there are a lot of places where it needs adding -- but if it works better than it did
previously and doesn't break i'll do merge and ask you to review afterwards if anything is
left out, unless you think i shouldn't.
    
    one question in PR #177 about logging would be handy to have your take on -- if rebind
fails will we still get decent output? -- apart from that i'm getting through it, things looking
reasonable so far (just tedious!)


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message