[jira] [Reopened] (BOOKKEEPER-718) AuditorLedgerCheckerTest is flakey

[Govind Menon (JIRA)]

 [ 
https://issues.apache.org/jira/browse/BOOKKEEPER-718?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Govind Menon reopened BOOKKEEPER-718:
-

> AuditorLedgerCheckerTest is flakey
> --
>
> Key: BOOKKEEPER-718
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-718
> Project: Bookkeeper
>  Issue Type: Bug
>  Components: bookkeeper-auto-recovery
>Reporter: Ivan Kelly
>Assignee: Govind Menon
> Fix For: 4.3.0
>
> Attachments: 
> 0001-BOOKKEEPER-718-AuditorLedgerCheckerTest-is-flakey.patch, 
> 0001-BOOKKEEPER-718-AuditorLedgerCheckerTest-is-flakey.patch, error505.txt, 
> error509.txt
>
>
> See:
> https://builds.apache.org/job/bookkeeper-trunk/505/
> & 
> https://builds.apache.org/job/bookkeeper-trunk/509
> Relevant errors attached.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Assigned] (BOOKKEEPER-718) AuditorLedgerCheckerTest is flakey

[Govind Menon (JIRA)]

 [ 
https://issues.apache.org/jira/browse/BOOKKEEPER-718?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Govind Menon reassigned BOOKKEEPER-718:
---

Assignee: Govind Menon  (was: Ivan Kelly)

> AuditorLedgerCheckerTest is flakey
> --
>
> Key: BOOKKEEPER-718
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-718
> Project: Bookkeeper
>  Issue Type: Bug
>  Components: bookkeeper-auto-recovery
>Reporter: Ivan Kelly
>Assignee: Govind Menon
> Fix For: 4.3.0
>
> Attachments: 
> 0001-BOOKKEEPER-718-AuditorLedgerCheckerTest-is-flakey.patch, 
> 0001-BOOKKEEPER-718-AuditorLedgerCheckerTest-is-flakey.patch, error505.txt, 
> error509.txt
>
>
> See:
> https://builds.apache.org/job/bookkeeper-trunk/505/
> & 
> https://builds.apache.org/job/bookkeeper-trunk/509
> Relevant errors attached.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868415#comment-15868415
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


Github user govind-menon commented on the issue:

https://github.com/apache/bookkeeper/pull/109
  
The current 2 test failures are being fixed in a separate JIRA and the 3rd 
test failure is known to be flakey - BOOKKEEPER-718


> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868409#comment-15868409
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


Github user govind-menon commented on the issue:

https://github.com/apache/bookkeeper/pull/109
  
The current test failures - 
org.apache.bookkeeper.client.BookieRecoveryTest.ensurePasswordUsedForOldLedgers[4]

org.apache.bookkeeper.client.BookieRecoveryTest.ensurePasswordUsedForOldLedgers[5]
 

will be fixed in another JIRA.


> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-998) Increased the max entry size to 5MB

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-998?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868400#comment-15868400
 ] 

ASF GitHub Bot commented on BOOKKEEPER-998:
---

Github user eolivelli commented on the issue:

https://github.com/apache/bookkeeper/pull/104
  
For me is +1 but I am not a committer. Usually @sijie merges PR or maybe 
@merlimat


> Increased the max entry size to 5MB
> ---
>
> Key: BOOKKEEPER-998
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-998
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Reporter: Robert Joseph Evans
>Assignee: Robert Joseph Evans
>
> In practice we have found the Bookkeeper can handle 5MB entries without 
> issue.  We should make the {{PerChannelBookieClient.MAX_FRAME_LENGTH}}, the 
> {{BookieNettyServer.maxMessageSize}} and the 
> {{EntryLogger.MAX_SANE_ENTRY_SIZE}} size all consistent with one another, and 
> ideally 5MB.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

[Apache Jenkins Server]

See 


Changes:

[govindappumenon] BOOKKEEPER-1004: Allow bookie garbage collection to be 
triggered

--
[...truncated 404 lines...]

---
 T E S T S
---
Running org.apache.bookkeeper.proto.TestBackwardCompatCMS42
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.573 sec - in 
org.apache.bookkeeper.proto.TestBackwardCompatCMS42
Running org.apache.bookkeeper.proto.TestDeathwatcher
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.34 sec - in 
org.apache.bookkeeper.proto.TestDeathwatcher
Running org.apache.bookkeeper.proto.NetworkLessBookieTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 13.495 sec - in 
org.apache.bookkeeper.proto.NetworkLessBookieTest
Running org.apache.bookkeeper.proto.TestBKStats
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.32 sec - in 
org.apache.bookkeeper.proto.TestBKStats
Running org.apache.bookkeeper.proto.TestPerChannelBookieClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.378 sec - in 
org.apache.bookkeeper.proto.TestPerChannelBookieClient
Running org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.331 sec - in 
org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Running org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.378 sec - in 
org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Running org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 36.139 sec - in 
org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Running org.apache.bookkeeper.bookie.BookieJournalTest
Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.465 sec - in 
org.apache.bookkeeper.bookie.BookieJournalTest
Running org.apache.bookkeeper.bookie.EntryLogTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.77 sec - in 
org.apache.bookkeeper.bookie.EntryLogTest
Running org.apache.bookkeeper.bookie.IndexCorruptionTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 66.487 sec - in 
org.apache.bookkeeper.bookie.IndexCorruptionTest
Running org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.572 sec - in 
org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Running org.apache.bookkeeper.bookie.TestSyncThread
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.159 sec - in 
org.apache.bookkeeper.bookie.TestSyncThread
Running org.apache.bookkeeper.bookie.TestLedgerDirsManager
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.689 sec - in 
org.apache.bookkeeper.bookie.TestLedgerDirsManager
Running org.apache.bookkeeper.bookie.BookieShutdownTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 26.106 sec - in 
org.apache.bookkeeper.bookie.BookieShutdownTest
Running org.apache.bookkeeper.bookie.UpdateCookieCmdTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.618 sec - in 
org.apache.bookkeeper.bookie.UpdateCookieCmdTest
Running org.apache.bookkeeper.bookie.UpgradeTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.61 sec - in 
org.apache.bookkeeper.bookie.UpgradeTest
Running org.apache.bookkeeper.bookie.TestGcOverreplicatedLedger
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.593 sec - in 
org.apache.bookkeeper.bookie.TestGcOverreplicatedLedger
Running org.apache.bookkeeper.bookie.CookieTest
Tests run: 17, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.762 sec - in 
org.apache.bookkeeper.bookie.CookieTest
Running org.apache.bookkeeper.bookie.BookieThreadTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.282 sec - in 
org.apache.bookkeeper.bookie.BookieThreadTest
Running org.apache.bookkeeper.bookie.BookieInitializationTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.922 sec - in 
org.apache.bookkeeper.bookie.BookieInitializationTest
Running org.apache.bookkeeper.bookie.LedgerCacheTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.976 sec - in 
org.apache.bookkeeper.bookie.LedgerCacheTest
Running org.apache.bookkeeper.bookie.CreateNewLogTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.905 sec - in 
org.apache.bookkeeper.bookie.CreateNewLogTest
Running org.apache.bookkeeper.bookie.CompactionTest
Tests run: 20, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 107.754 sec - 
in org.apache.bookkeeper.bookie.CompactionTest
Running org.apache.bookkeeper.metastore.TestMetaStore
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.535 sec - in 
org.apache.bookkeeper.metas

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868304#comment-15868304
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


GitHub user govind-menon reopened a pull request:

https://github.com/apache/bookkeeper/pull/109

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manu…

Ran CompactionTest#testForceGarbageCollection

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/govind-menon/bookkeeper BOOKKEEPER-1004

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/bookkeeper/pull/109.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #109


commit 68688fac643ab991873130780703172890891a15
Author: Govind Menon 
Date:   2017-02-14T18:06:44Z

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manually 
from tests




> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868303#comment-15868303
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


Github user govind-menon closed the pull request at:

https://github.com/apache/bookkeeper/pull/109


> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-998) Increased the max entry size to 5MB

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-998?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868210#comment-15868210
 ] 

ASF GitHub Bot commented on BOOKKEEPER-998:
---

Github user revans2 commented on the issue:

https://github.com/apache/bookkeeper/pull/104
  
@eolivelli and @jvrao is there anything more I need to do to get this 
merged in?


> Increased the max entry size to 5MB
> ---
>
> Key: BOOKKEEPER-998
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-998
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Reporter: Robert Joseph Evans
>Assignee: Robert Joseph Evans
>
> In practice we have found the Bookkeeper can handle 5MB entries without 
> issue.  We should make the {{PerChannelBookieClient.MAX_FRAME_LENGTH}}, the 
> {{BookieNettyServer.maxMessageSize}} and the 
> {{EntryLogger.MAX_SANE_ENTRY_SIZE}} size all consistent with one another, and 
> ideally 5MB.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

[Apache Jenkins Server]

See 


Changes:

[govindappumenon] BOOKKEEPER-1004: Allow bookie garbage collection to be 
triggered

--
[...truncated 410 lines...]
Running org.apache.bookkeeper.proto.TestDeathwatcher
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.391 sec - in 
org.apache.bookkeeper.proto.TestDeathwatcher
Running org.apache.bookkeeper.proto.NetworkLessBookieTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 13.293 sec - in 
org.apache.bookkeeper.proto.NetworkLessBookieTest
Running org.apache.bookkeeper.proto.TestBKStats
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.309 sec - in 
org.apache.bookkeeper.proto.TestBKStats
Running org.apache.bookkeeper.proto.TestPerChannelBookieClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.845 sec - in 
org.apache.bookkeeper.proto.TestPerChannelBookieClient
Running org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.335 sec - in 
org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Running org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.337 sec - in 
org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Running org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 36.391 sec - in 
org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Running org.apache.bookkeeper.bookie.BookieJournalTest
Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.59 sec - in 
org.apache.bookkeeper.bookie.BookieJournalTest
Running org.apache.bookkeeper.bookie.EntryLogTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.889 sec - in 
org.apache.bookkeeper.bookie.EntryLogTest
Running org.apache.bookkeeper.bookie.IndexCorruptionTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 66.667 sec - in 
org.apache.bookkeeper.bookie.IndexCorruptionTest
Running org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.552 sec - in 
org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Running org.apache.bookkeeper.bookie.TestSyncThread
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.046 sec - in 
org.apache.bookkeeper.bookie.TestSyncThread
Running org.apache.bookkeeper.bookie.TestLedgerDirsManager
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.597 sec - in 
org.apache.bookkeeper.bookie.TestLedgerDirsManager
Running org.apache.bookkeeper.bookie.BookieShutdownTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 24.406 sec - in 
org.apache.bookkeeper.bookie.BookieShutdownTest
Running org.apache.bookkeeper.bookie.UpdateCookieCmdTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.959 sec - in 
org.apache.bookkeeper.bookie.UpdateCookieCmdTest
Running org.apache.bookkeeper.bookie.UpgradeTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.993 sec - in 
org.apache.bookkeeper.bookie.UpgradeTest
Running org.apache.bookkeeper.bookie.TestGcOverreplicatedLedger
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.668 sec - in 
org.apache.bookkeeper.bookie.TestGcOverreplicatedLedger
Running org.apache.bookkeeper.bookie.CookieTest
Tests run: 17, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.6 sec - in 
org.apache.bookkeeper.bookie.CookieTest
Running org.apache.bookkeeper.bookie.BookieThreadTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.294 sec - in 
org.apache.bookkeeper.bookie.BookieThreadTest
Running org.apache.bookkeeper.bookie.BookieInitializationTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.592 sec - in 
org.apache.bookkeeper.bookie.BookieInitializationTest
Running org.apache.bookkeeper.bookie.LedgerCacheTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.738 sec - in 
org.apache.bookkeeper.bookie.LedgerCacheTest
Running org.apache.bookkeeper.bookie.CreateNewLogTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.917 sec - in 
org.apache.bookkeeper.bookie.CreateNewLogTest
Running org.apache.bookkeeper.bookie.CompactionTest
Tests run: 20, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 108.263 sec - 
in org.apache.bookkeeper.bookie.CompactionTest
Running org.apache.bookkeeper.metastore.TestMetaStore
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.566 sec - in 
org.apache.bookkeeper.metastore.TestMetaStore
Running org.apache.bookkeeper.auth.TestAuth
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 26.376 sec - 
in org.apache.bookkeeper.auth.TestAuth
Running org.apache.bookkeeper.util.TestDiskChecker
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.389 sec - in 
org

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868058#comment-15868058
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


Github user govind-menon closed the pull request at:

https://github.com/apache/bookkeeper/pull/109


> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868060#comment-15868060
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


GitHub user govind-menon reopened a pull request:

https://github.com/apache/bookkeeper/pull/109

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manu…

Ran CompactionTest#testForceGarbageCollection

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/govind-menon/bookkeeper BOOKKEEPER-1004

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/bookkeeper/pull/109.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #109


commit 68688fac643ab991873130780703172890891a15
Author: Govind Menon 
Date:   2017-02-14T18:06:44Z

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manually 
from tests




> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

[Apache Jenkins Server]

See 


Changes:

[govindappumenon] BOOKKEEPER-1004: Allow bookie garbage collection to be 
triggered

--
[...truncated 373 lines...]
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ bookkeeper-server ---
[INFO] Deleting 

 (includes = [dependency-reduced-pom.xml], excludes = [])
[INFO] 
[INFO] --- apache-rat-plugin:0.7:check (default-cli) @ bookkeeper-server ---
[INFO] Exclude: **/DataFormats.java
[INFO] Exclude: **/BookkeeperProtocol.java
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.1:process (default) @ 
bookkeeper-server ---
[INFO] 
[INFO] --- maven-resources-plugin:2.4.3:resources (default-resources) @ 
bookkeeper-server ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.5.1:compile (default-compile) @ 
bookkeeper-server ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 244 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.4.3:testResources (default-testResources) @ 
bookkeeper-server ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.5.1:testCompile (default-testCompile) @ 
bookkeeper-server ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 114 source files to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.19.1:test (default-test) @ bookkeeper-server 
---
[WARNING] The parameter forkMode is deprecated since version 2.14. Use 
forkCount and reuseForks instead.

---
 T E S T S
---
Running org.apache.bookkeeper.proto.TestBackwardCompatCMS42
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.775 sec - in 
org.apache.bookkeeper.proto.TestBackwardCompatCMS42
Running org.apache.bookkeeper.proto.TestDeathwatcher
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.207 sec - in 
org.apache.bookkeeper.proto.TestDeathwatcher
Running org.apache.bookkeeper.proto.NetworkLessBookieTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 14.772 sec - in 
org.apache.bookkeeper.proto.NetworkLessBookieTest
Running org.apache.bookkeeper.proto.TestBKStats
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.318 sec - in 
org.apache.bookkeeper.proto.TestBKStats
Running org.apache.bookkeeper.proto.TestPerChannelBookieClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.3 sec - in 
org.apache.bookkeeper.proto.TestPerChannelBookieClient
Running org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.509 sec - in 
org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest
Running org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.369 sec - in 
org.apache.bookkeeper.conf.SystemPropertiesConfigurationTest
Running org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 37.943 sec - in 
org.apache.bookkeeper.zookeeper.TestZooKeeperClient
Running org.apache.bookkeeper.bookie.BookieJournalTest
Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.51 sec - in 
org.apache.bookkeeper.bookie.BookieJournalTest
Running org.apache.bookkeeper.bookie.EntryLogTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.468 sec - in 
org.apache.bookkeeper.bookie.EntryLogTest
Running org.apache.bookkeeper.bookie.IndexCorruptionTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 66.129 sec - in 
org.apache.bookkeeper.bookie.IndexCorruptionTest
Running org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.326 sec - in 
org.apache.bookkeeper.bookie.IndexPersistenceMgrTest
Running org.apache.bookkeeper.bookie.TestSyncThread
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.278 sec - in 
org.apache.bookkeeper.bookie.TestSyncThread
Running org.apache.bookkeeper.bookie.TestLedgerDirsManager
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.935 sec - in 
org.apache.bookkeeper.bookie.TestLedgerDirsManager
Running org.apache.bookkeeper.bookie.BookieShutdownTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 24.768 sec - in 
org.apache.bookkeeper.bookie.Bo

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867988#comment-15867988
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


Github user govind-menon closed the pull request at:

https://github.com/apache/bookkeeper/pull/109


> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867989#comment-15867989
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1004:


GitHub user govind-menon reopened a pull request:

https://github.com/apache/bookkeeper/pull/109

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manu…

Ran CompactionTest#testForceGarbageCollection

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/govind-menon/bookkeeper BOOKKEEPER-1004

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/bookkeeper/pull/109.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #109


commit 68688fac643ab991873130780703172890891a15
Author: Govind Menon 
Date:   2017-02-14T18:06:44Z

BOOKKEEPER-1004: Allow bookie garbage collection to be triggered manually 
from tests




> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BOOKKEEPER-1004) Allow bookie garbage collection to be triggered manually from tests

[Govind Menon (JIRA)]

[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867986#comment-15867986
 ] 

Govind Menon commented on BOOKKEEPER-1004:
--

[~eolivelli] [~mmerli] - tagging for review (let me know if I should bug 
someone else)

> Allow bookie garbage collection to be triggered manually from tests
> ---
>
> Key: BOOKKEEPER-1004
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1004
> Project: Bookkeeper
>  Issue Type: Improvement
>  Components: bookkeeper-server
>Affects Versions: 4.5.0, 4.6.0
>Reporter: Govind Menon
>Priority: Minor
>  Labels: test
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The current gc tests rely on waiting on a timeout for gc to run. It's
> never certain whether it has run or not or if it's still running. 
> This patch allows tests to trigger a gc run and gives the client
> a future to know when it has completed. The gc algorithm is unchangedI but 
> now it runs in a scheduled executor rather than as a
> Thread.
> This work was originally done by Ivan Kelly and I am just pushing it back to 
> open source



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Build failed in Jenkins: bookkeeper-master #1659

[Apache Jenkins Server]

See 

--
[...truncated 375 lines...]
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 114 source files to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.19.1:test (default-test) @ bookkeeper-server 
---
[WARNING] The parameter forkMode is deprecated since version 2.14. Use 
forkCount and reuseForks instead.

---
 T E S T S
---
Running org.apache.bookkeeper.test.BookieClientTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.872 sec - in 
org.apache.bookkeeper.test.BookieClientTest
Running org.apache.bookkeeper.test.BookieReadWriteTest
Tests run: 104, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 33.283 sec - 
in org.apache.bookkeeper.test.BookieReadWriteTest
Running org.apache.bookkeeper.test.AsyncLedgerOpsTest
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.194 sec - in 
org.apache.bookkeeper.test.AsyncLedgerOpsTest
Running org.apache.bookkeeper.test.CloseTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 5.505 sec - in 
org.apache.bookkeeper.test.CloseTest
Running org.apache.bookkeeper.test.ConcurrentLedgerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.479 sec - in 
org.apache.bookkeeper.test.ConcurrentLedgerTest
Running org.apache.bookkeeper.test.LedgerDeleteTest
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 29.727 sec - in 
org.apache.bookkeeper.test.LedgerDeleteTest
Running org.apache.bookkeeper.test.ForceReadOnlyBookieTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.034 sec - in 
org.apache.bookkeeper.test.ForceReadOnlyBookieTest
Running org.apache.bookkeeper.test.TestBackwardCompat
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 19.339 sec - in 
org.apache.bookkeeper.test.TestBackwardCompat
Running org.apache.bookkeeper.test.ReadOnlyBookieTest
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.485 sec - in 
org.apache.bookkeeper.test.ReadOnlyBookieTest
Running org.apache.bookkeeper.test.MultipleThreadReadTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.664 sec - in 
org.apache.bookkeeper.test.MultipleThreadReadTest
Running org.apache.bookkeeper.test.LedgerCreateDeleteTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 5.297 sec - in 
org.apache.bookkeeper.test.LedgerCreateDeleteTest
Running org.apache.bookkeeper.test.ConfigurationTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.228 sec - in 
org.apache.bookkeeper.test.ConfigurationTest
Running org.apache.bookkeeper.test.BookieFailureTest
Tests run: 64, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 30.539 sec - 
in org.apache.bookkeeper.test.BookieFailureTest
Running org.apache.bookkeeper.test.ConditionalSetTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.394 sec - in 
org.apache.bookkeeper.test.ConditionalSetTest
Running org.apache.bookkeeper.test.BookieJournalRollingTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 42.879 sec - in 
org.apache.bookkeeper.test.BookieJournalRollingTest
Running org.apache.bookkeeper.test.BookieZKExpireTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 16.034 sec - in 
org.apache.bookkeeper.test.BookieZKExpireTest
Running org.apache.bookkeeper.client.TestSpeculativeRead
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 66.657 sec - in 
org.apache.bookkeeper.client.TestSpeculativeRead
Running org.apache.bookkeeper.client.TestBookieHealthCheck
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 17.224 sec - in 
org.apache.bookkeeper.client.TestBookieHealthCheck
Running org.apache.bookkeeper.client.RoundRobinDistributionScheduleTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.189 sec - in 
org.apache.bookkeeper.client.RoundRobinDistributionScheduleTest
Running org.apache.bookkeeper.client.UpdateLedgerCmdTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 10.318 sec - in 
org.apache.bookkeeper.client.UpdateLedgerCmdTest
Running org.apache.bookkeeper.client.TestLedgerFragmentReplication
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.264 sec - in 
org.apache.bookkeeper.client.TestLedgerFragmentReplication
Running org.apache.bookkeeper.client.BookieRecoveryTest
Tests run: 96, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 74.538 sec <<< 
FAILURE! - in org.apache.bookkeeper.client.BookieRecoveryTest
ensurePasswordUsedForOldLedgers[4](org.apache.bookkeeper.client.BookieRecoveryTest)
  Time elapsed: 2.343 sec  <<< ERROR!
org.apache.bookkeeper.client.BKException$BKNoSuchLedgerExistsException

ensurePasswordUsedForOldLedgers[5](org.apache.bookkeeper.client.BookieRecoveryTest