GitHub user sijie opened a pull request:

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

    BOOKKEEPER-748: Move fence requests out of read threads

    This change is moving the fence request out of current read threads and 
using callback to trigger scheduling read entry and also it refactors the 
ReadEntryProcessV3 to support long poll in the subsequent requests.
    
    Major changes:
    
    - change fence request to use SettableFuture
    - refactor ReadEntryProcessV3: to support run fence request in callback and 
support long poll (in subsequent requests)
    - fix stats issue: requests stats measure the latency from bookie receives 
the request until it sends the response.
    - remove "public static final" for state fields. it is not needed for 
variables in Interface.

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

    $ git pull https://github.com/sijie/bookkeeper 
longpoll/part1_server_side_change

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

    https://github.com/apache/bookkeeper/pull/174.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 #174
    
----
commit 56f6824a6f57360fbd999995d9959a2966046183
Author: Robin Dhamankar <rdhaman...@twitter.com>
Date:   2014-02-12T22:50:47Z

    BookKeeper: Stats for pending tasks on server thread pools
    - Stats for pending tasks on server thread pools
    
    RB_ID=286873

commit 8364c12d705b1eabae394807419a9989a9608f56
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T07:37:24Z

    Improvements on OrderedSafeExecutor
    
    - use listeningscheduledexecutorservice for the threads
    - as a general util class, expose chooseThread to allow applications use 
specific thread
    - add a method to force shutdown executor

commit ac8431c4721498f5fd7c6d387a09039652c47f9f
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T09:50:18Z

    Long Poll (Part 2) - Run fence request in a separated thread

commit d43400d3e42057b938dc6b31b1761004da1f963b
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T09:51:35Z

    Merge branch 'master' into add_monitored_thread_pool

commit c1a02b3d371c93eaf4d8ff14ee3e4f669ab09874
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T09:51:58Z

    Merge branch 'add_monitored_thread_pool' into 
longpoll/part1_server_side_change

commit c57fefdae1de8eec6e44a2af692fb6fe07224abd
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T10:34:22Z

    BOOKKEEPER-748: Move fence requests out of read threads

commit d431c643c7feb78045d9478988fe8fbacc7ed703
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T11:01:38Z

    Merge the change for fence request

commit 4e14991a28cd30bca0e42803c49eafc39b11fc60
Author: Sijie Guo <si...@apache.org>
Date:   2017-06-01T11:02:17Z

    Merge remote-tracking branch 'apache/master' into 
longpoll/part1_server_side_change

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to