Github user franckcuny commented on the issue:
https://github.com/apache/incubator-distributedlog/pull/49
Got it. LGTM.
---
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 an
Github user sijie commented on the issue:
https://github.com/apache/incubator-distributedlog/pull/49
@franckcuny yup. the change is to verify the pull request pass the tests. I
think we need a coverall token to report coverall reports. I am thinking of
moving the coverall report to be
Github user franckcuny commented on a diff in the pull request:
https://github.com/apache/incubator-distributedlog/pull/49#discussion_r91756413
--- Diff: .travis.yml ---
@@ -25,11 +33,7 @@ before_install:
script:
- travis_retry mvn --batch-mode clean apache-rat:ch
[
https://issues.apache.org/jira/browse/DL-120?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15734879#comment-15734879
]
Jia Zhai commented on DL-120:
-
There is a discussion in the dev mail thread. will update here once
Thanks for the suggestions. would like to collect some suggestions and
feedbacks here, then update the tickets. And will update the ticket at next
Wensday, Is this OK?
On Fri, Dec 9, 2016 at 1:01 PM, Sijie Guo wrote:
> Good suggestions, Xi.
>
> Jia, do you want to update draft and then update th