[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997858#comment-15997858 ] ASF GitHub Bot commented on BOOKKEEPER-1008: Github user eolivelli commen

[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-04 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/116 @kishorekasi it is great to see all tests now passing ! @kishorekasi did you take a look to latest @merlimat commit on this issue ? @kishorekasi @sijie @revans2 Now that all th

[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-04 Thread kishorekasi
Github user kishorekasi commented on the issue: https://github.com/apache/bookkeeper/pull/116 @eolivelli @sijie @merlimat With my latest commit this evening, all tests seem to pass now. Let me know if you are seeing any test failures in your local runs. --- If your project is set up

[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997852#comment-15997852 ] ASF GitHub Bot commented on BOOKKEEPER-1008: Github user kishorekasi comm

Jenkins build is back to normal : bookkeeper-master-git-pullrequest #341

2017-05-04 Thread Apache Jenkins Server
See

Build failed in Jenkins: bookkeeper-master-git-pullrequest #340

2017-05-04 Thread Apache Jenkins Server
See Changes: [kudayashankar] BOOKKEEPER-1008 Move to Netty4.1 [kudayashankar] Addressing review comments - Use Netty 4.1.9 - Do not include classifier [kudayashankar] Addressing ref count issue

Build failed in Jenkins: bookkeeper-master-git-pullrequest #339

2017-05-04 Thread Apache Jenkins Server
See Changes: [kudayashankar] BOOKKEEPER-1008 Move to Netty4.1 [kudayashankar] Addressing review comments - Use Netty 4.1.9 - Do not include classifier [kudayashankar] Addressing ref count issue

[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997663#comment-15997663 ] ASF GitHub Bot commented on BOOKKEEPER-1008: GitHub user merlimat opened

[GitHub] bookkeeper pull request #138: BOOKKEEPER-1008: Netty 4.1

2017-05-04 Thread merlimat
GitHub user merlimat opened a pull request: https://github.com/apache/bookkeeper/pull/138 BOOKKEEPER-1008: Netty 4.1 Added more ref-count fixes from yahoo-4.3 branch on top of #116 You can merge this pull request into a Git repository by running: $ git pull https://github.com/

[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997654#comment-15997654 ] ASF GitHub Bot commented on BOOKKEEPER-1008: Github user merlimat comment

[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-04 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/bookkeeper/pull/116 @kishorekasi I'll push my branch in a new PR to trigger Jenkins with my changes as well --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997648#comment-15997648 ] ASF GitHub Bot commented on BOOKKEEPER-1008: Github user merlimat comment

[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-04 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/bookkeeper/pull/116 @kishorekasi I have fixed the issues with the `BookieFailureTest`. I have applied multiple ref-count fixes as well from the yahoo branch.. pushed to https://github.com/merlimat/bookkeeper/commit

Build failed in Jenkins: bookkeeper-master-git-pullrequest #338

2017-05-04 Thread Apache Jenkins Server
See Changes: [kudayashankar] BOOKKEEPER-1008 Move to Netty4.1 [kudayashankar] Addressing review comments - Use Netty 4.1.9 - Do not include classifier [kudayashankar] Addressing ref count issue

[jira] [Resolved] (BOOKKEEPER-1046) Avoid long to Long conversion in OrderedSafeExecutor task submit

2017-05-04 Thread Matteo Merli (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1046?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matteo Merli resolved BOOKKEEPER-1046. -- Resolution: Fixed > Avoid long to Long conversion in OrderedSafeExecutor task submi

Build failed in Jenkins: bookkeeper-master-git-pullrequest #337

2017-05-04 Thread Apache Jenkins Server
See Changes: [kudayashankar] BOOKKEEPER-1008 Move to Netty4.1 [kudayashankar] Addressing review comments - Use Netty 4.1.9 - Do not include classifier [kudayashankar] Addressing ref count issue

[jira] [Commented] (BOOKKEEPER-1047) Add missing error code in ZK setData return path

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997236#comment-15997236 ] ASF GitHub Bot commented on BOOKKEEPER-1047: Github user asfgit closed th

[GitHub] bookkeeper pull request #137: BOOKKEEPER-1047: Add missing error code in ZK ...

2017-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/bookkeeper/pull/137 --- 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] bookkeeper pull request #136: BOOKKEEPER-1046: Avoid long to Long conversion...

2017-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/bookkeeper/pull/136 --- 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

[jira] [Commented] (BOOKKEEPER-1046) Avoid long to Long conversion in OrderedSafeExecutor task submit

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997223#comment-15997223 ] ASF GitHub Bot commented on BOOKKEEPER-1046: Github user asfgit closed th

Build failed in Jenkins: bookkeeper-master-git-pullrequest #336

2017-05-04 Thread Apache Jenkins Server
See Changes: [mmerli] BOOKKEEPER-1047: Add missing error code in ZK setData return path -- [...truncated 24.37 KB...] Tests run: 4, Failures: 0, Errors: 0,

[jira] [Commented] (BOOKKEEPER-1008) Move to netty4

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997132#comment-15997132 ] ASF GitHub Bot commented on BOOKKEEPER-1008: Github user merlimat comment

[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-04 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/bookkeeper/pull/116 @kishorekasi can you also rebase on top of last master? This should reduce the count of tests failing --- If your project is set up for it, you can reply to this email and have your reply appea

[jira] [Commented] (BOOKKEEPER-1047) Add missing error code in ZK setData return path

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997112#comment-15997112 ] ASF GitHub Bot commented on BOOKKEEPER-1047: GitHub user merlimat opened

[GitHub] bookkeeper pull request #137: BOOKKEEPER-1047: Add missing error code in ZK ...

2017-05-04 Thread merlimat
GitHub user merlimat opened a pull request: https://github.com/apache/bookkeeper/pull/137 BOOKKEEPER-1047: Add missing error code in ZK setData return path The log warning is not printing the error code returned by ZooKeeper You can merge this pull request into a Git repository by

[jira] [Created] (BOOKKEEPER-1047) Add missing error code in ZK setData return path

2017-05-04 Thread Matteo Merli (JIRA)
Matteo Merli created BOOKKEEPER-1047: Summary: Add missing error code in ZK setData return path Key: BOOKKEEPER-1047 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1047 Project: Bookkeeper

[jira] [Commented] (BOOKKEEPER-1046) Avoid long to Long conversion in OrderedSafeExecutor task submit

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15997042#comment-15997042 ] ASF GitHub Bot commented on BOOKKEEPER-1046: Github user merlimat comment

[GitHub] bookkeeper issue #136: BOOKKEEPER-1046: Avoid long to Long conversion in Ord...

2017-05-04 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/bookkeeper/pull/136 > can you drop the old method as it is no more used ? @eolivelli We still use it in Pulsar/ManagedLedger to have tasks in the executor executed in order by topic name --- If your proj

[jira] [Commented] (BOOKKEEPER-1045) Execute tests in different JVM processes

2017-05-04 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1045?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996752#comment-15996752 ] Hudson commented on BOOKKEEPER-1045: SUCCESS: Integrated in Jenkins build bookkee

Jenkins build is back to normal : bookkeeper-master #1737

2017-05-04 Thread Apache Jenkins Server
See

[jira] [Commented] (BOOKKEEPER-1046) Avoid long to Long conversion in OrderedSafeExecutor task submit

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996431#comment-15996431 ] ASF GitHub Bot commented on BOOKKEEPER-1046: Github user eolivelli commen

[jira] [Commented] (BOOKKEEPER-1046) Avoid long to Long conversion in OrderedSafeExecutor task submit

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996432#comment-15996432 ] ASF GitHub Bot commented on BOOKKEEPER-1046: Github user eolivelli commen

[GitHub] bookkeeper issue #136: BOOKKEEPER-1046: Avoid long to Long conversion in Ord...

2017-05-04 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/136 Overall is OK for me, one little change can you drop the old method as it is no more used ? `public void submitOrdered(Object orderingKey, SafeRunnable r) {` BK code does no

[GitHub] bookkeeper issue #136: BOOKKEEPER-1046: Avoid long to Long conversion in Ord...

2017-05-04 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/136 All JUnit tests are passing --- 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

Build failed in Jenkins: bookkeeper-master-git-pullrequest #335

2017-05-04 Thread Apache Jenkins Server
See Changes: [mmerli] BOOKKEEPER-1046: Avoid long to Long conversion in OrderedSafeExecutor -- [...truncated 157.41 KB...] Running org.apache.bookkeeper.me

[jira] [Commented] (BOOKKEEPER-1045) Execute tests in different JVM processes

2017-05-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1045?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996341#comment-15996341 ] ASF GitHub Bot commented on BOOKKEEPER-1045: Github user asfgit closed th

[GitHub] bookkeeper pull request #135: BOOKKEEPER-1045: Execute tests in different JV...

2017-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/bookkeeper/pull/135 --- 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