[GitHub] kafka pull request: KAFKA-3651; awaitFlushCompletion() in RecordAc...

2016-05-03 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/1315

KAFKA-3651; awaitFlushCompletion() in RecordAccumulator should always 
decrement flushesInProgress count



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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-3655

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

https://github.com/apache/kafka/pull/1315.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 #1315


commit e7533f57cc0169ba0c082ef6c9cc9ddefd902e85
Author: Chen Zhu <amandazhu19620...@gmail.com>
Date:   2016-05-04T00:29:56Z

KAFKA-3651; awaitFlushCompletion() in RecordAccumulator should always 
decrement flushesInProgress count




---
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.
---


[GitHub] kafka pull request: KAFKA-3383: remove in flight request only afte...

2016-05-03 Thread zhuchen1018
Github user zhuchen1018 closed the pull request at:

https://github.com/apache/kafka/pull/1050


---
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.
---


[GitHub] kafka pull request: KAFKA-3648: maxTimeToBlock should be enforced

2016-05-02 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/1304

KAFKA-3648: maxTimeToBlock should be enforced

maxTimeToBlock in allocate(int size, long maxTimeToBlock) is enforced. 

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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-3648

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

https://github.com/apache/kafka/pull/1304.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 #1304


commit c63bcf50826d740d1da04b54da2ded7427da57f7
Author: Chen Zhu <amandazhu19620...@gmail.com>
Date:   2016-05-03T00:08:51Z

KAFKA-3648:maxTimeToBlock should be enforced




---
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.
---


[GitHub] kafka pull request: KAFKA-3383: remove in flight request only afte...

2016-03-19 Thread zhuchen1018
GitHub user zhuchen1018 reopened a pull request:

https://github.com/apache/kafka/pull/1050

KAFKA-3383: remove in flight request only after response parsing succeeds

@becketqin, could you take a look at the patch?

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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-3383

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

https://github.com/apache/kafka/pull/1050.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 #1050


commit 84372bbe06bd121e758348b54d2d79d5fb4fd095
Author: Chen Zhu <amandazhu19620...@gmail.com>
Date:   2016-03-11T03:43:52Z

KAFKA-3383: remove in flight request only after response parsing succeeds




---
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.
---


[GitHub] kafka pull request: KAFKA-3383: remove in flight request only afte...

2016-03-19 Thread zhuchen1018
Github user zhuchen1018 closed the pull request at:

https://github.com/apache/kafka/pull/1050


---
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.
---


[GitHub] kafka pull request: KAFKA-3383: remove in flight request only afte...

2016-03-10 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/1050

KAFKA-3383: remove in flight request only after response parsing succeeds

@becketqin, could you take a look at the patch?

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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-3383

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

https://github.com/apache/kafka/pull/1050.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 #1050


commit 84372bbe06bd121e758348b54d2d79d5fb4fd095
Author: Chen Zhu <amandazhu19620...@gmail.com>
Date:   2016-03-11T03:43:52Z

KAFKA-3383: remove in flight request only after response parsing succeeds




---
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.
---


[GitHub] kafka pull request: KAFKA-3257: disable bootstrap-test-env.sh --co...

2016-02-24 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/969

KAFKA-3257: disable bootstrap-test-env.sh --colour option

@becketqin, when you get a chance, could you take a look at the patch? 

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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-3257

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

https://github.com/apache/kafka/pull/969.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 #969


commit 00d99e751e938c82dd9791de74a00634e2f35418
Author: zhuchen1018 <amandazhu19620...@gmail.com>
Date:   2016-02-25T04:35:06Z

KAFKA-3257: disable bootstrap-test-env.sh --colour option




---
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.
---


[GitHub] kafka pull request: MINOR - remove unused imports in package kafka...

2016-02-18 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/935

MINOR - remove unused imports in package kafka.utils



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

$ git pull https://github.com/zhuchen1018/kafka minor-remove-unused-imports

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

https://github.com/apache/kafka/pull/935.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 #935


commit b979a44afec4285481dbf9c255feb32e0b107620
Author: zhuchen1018 <amandazhu19620...@gmail.com>
Date:   2016-02-13T06:17:10Z

KAFKA-2757; Consolidate BrokerEndPoint and EndPoint

commit ef58a1aa32300d7837d96667cca13fdea4693f06
Author: zhuchen1018 <amandazhu19620...@gmail.com>
Date:   2016-02-19T03:02:34Z

minor-remove-unused-imports




---
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.
---


[GitHub] kafka pull request: KAFKA-2757; Consolidate BrokerEndPoint and End...

2016-02-12 Thread zhuchen1018
GitHub user zhuchen1018 opened a pull request:

https://github.com/apache/kafka/pull/911

KAFKA-2757; Consolidate BrokerEndPoint and EndPoint



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

$ git pull https://github.com/zhuchen1018/kafka KAFKA-2757

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

https://github.com/apache/kafka/pull/911.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 #911


commit b979a44afec4285481dbf9c255feb32e0b107620
Author: zhuchen1018 <amandazhu19620...@gmail.com>
Date:   2016-02-13T06:17:10Z

KAFKA-2757; Consolidate BrokerEndPoint and EndPoint




---
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.
---