[GitHub] kafka pull request: KAFKA-2934: Offset storage file configuration ...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/734 KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig Added offsetBackingStore config to StandaloneConfig and DistributedConfig; Added config for offset.storage.topic and config.storage.topic into DistributedConfig; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2934 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/734.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 #734 commit 8a59426b2d348826584e8a618676b739dd7918d7 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-21T08:34:27Z KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig commit 54bc2bcf5259cba9d99c74566d57823784065ab9 Author: jinxing <jinx...@fenbi.com> Date: 2016-03-02T06:56:04Z mod commit b491755fe3ea3e3420cd0b2b4297ca3196e89307 Author: jinxing <jinx...@fenbi.com> Date: 2016-03-02T07:03:25Z fix KakfaStatusBackingStore --- 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-2934: Offset storage file configuration ...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/734 --- 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-2934: Offset storage file configuration ...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/734 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix commit 1e42f33e835e75e0a27a41aa7b4b35e86cf3d457 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T12:45:02Z KAFKA-2875:wip commit 14f74f56481074acc2366fcd9f77f653c9f09ba6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T13:49:03Z KAFKA-2875: fixes commit 69c9e06e8ec405834b8cdbab69672176c520ac2b Author: jinxing <jinx...@fenbi.com> Date: 2016-02-15T02:57:14Z KAFKA-2875: fix --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix commit 1e42f33e835e75e0a27a41aa7b4b35e86cf3d457 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T12:45:02Z KAFKA-2875:wip commit 14f74f56481074acc2366fcd9f77f653c9f09ba6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T13:49:03Z KAFKA-2875: fixes commit 69c9e06e8ec405834b8cdbab69672176c520ac2b Author: jinxing <jinx...@fenbi.com> Date: 2016-02-15T02:57:14Z KAFKA-2875: fix --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix commit 1e42f33e835e75e0a27a41aa7b4b35e86cf3d457 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T12:45:02Z KAFKA-2875:wip commit 14f74f56481074acc2366fcd9f77f653c9f09ba6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-14T13:49:03Z KAFKA-2875: fixes --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin commit fbd380659727d991dff242be33cc6a3bb78f4861 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-28T06:28:25Z Merge pull request #15 from apache/trunk 2016-01-28 commit f21aa55ed68907376d5b0924e228875530cc1046 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:10:30Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution (merge to trunk and resolve conflict) commit 51fcc408302ebb0c4adaf2a4d0e6647cc469c6a0 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-28T07:43:52Z added a new line commit 8a6cbad74ca4f07a4c70c1d522b604d58e4917c6 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T08:49:06Z KAFKA-2875: create deduplicated dependant-libs and use symlink to construct classpath commit 153a1177c943e76c9c8457c47244ec59ea91d6fc Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T09:42:37Z small fix commit 1d283120bd7c3b90928090c4d22376d4ac05c4d5 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:09:46Z KAFKA-2875: modify classpath in windows bat commit 29c1797ae4f3ba47445e45049c8fc0fc2e1609f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T10:13:20Z mod server.properties for test commit a1993e5ca2908862340113ce965bd7fdc5020bab Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T12:44:22Z KAFKA-2875: small fix commit e523bd2ce91e03e38c20413aef3c48998fc3c263 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-01T16:12:27Z KAFKA-2875: small fix commit fb27bdeba925e6833ca9bc9feb1d6d3cf55c5aaf Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:44:32Z KAFKA-2875: replace PROJECT_NAMES with PROJECT_NAME, use the central deduplicated libs if PROJECT_NAME not specified commit f8ba5a920a0db8654c0776ad8449b167689c0eb4 Author: jinxing <jinx...@fenbi.com> Date: 2016-02-02T09:48:00Z small fix --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-3054: Connect Herder fail if sent a wron...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/782 KAFKA-3054: Connect Herder fail if sent a wrong connector config or task config, catch exceptions to guard The exceptions may propagate to DistributedHerder when start or stop connectors or tasks; The best solution is to track the track the status of connectors and tasks; This patch catch exceptions to guard the DistributedHerder; this a short-term solution; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-3054 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/782.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 #782 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit e75c8bbb3329868f92becc86645d31fa21c6c7f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-17T03:11:45Z KAFKA-3054: Connect Herder fail if sent a wrong connector config or task config, catch exceptions to guard commit efd6023821df014b06e509c1a13fd4e9d08113cb Author: jinxing <jinx...@fenbi.com> Date: 2016-01-25T13:20:36Z catch the exception thrown during stopping connector or task and try to restart commit d05af92d44ed26001f0ec96f5376790d2009249e Author: jinxing <jinx...@fenbi.com> Date: 2016-01-25T13:22:45Z small fix commit b6521f7d90b8fabfe3bfd086c188977d444660b5 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-25T13:23:12Z small fix --- 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-3054: Connect Herder fail if sent a wron...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/782 --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage commit 320386d2c484dac7eed9b7fe1584ca376e4ad897 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T06:09:00Z fix commit 67d1b21661886b204bbb86bc472a2ecce57613dc Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T07:45:53Z small fix commit c31562642eaf2fa9bd35366246430fa6de8be6d8 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:46:43Z Revert "small fix" This reverts commit 67d1b21661886b204bbb86bc472a2ecce57613dc. commit ef1d9a8921d2e5060824878d7212b1d50454e0f9 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:48:44Z Revert "fix" This reverts commit 320386d2c484dac7eed9b7fe1584ca376e4ad897. commit 5c09181de11a5cdae78acf690513f0166a1d5c21 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:48:58Z Revert "KAFKA-2944: fix NullPointerException in KafkaConfigStorage" This reverts commit 82150dccc59ac0e436acc5186d2b8fb66c9df671. commit eba2a0737ae93f3d8e0137c5901277f53c62daac Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T08:25:36Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage commit 320386d2c484dac7eed9b7fe1584ca376e4ad897 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T06:09:00Z fix commit 67d1b21661886b204bbb86bc472a2ecce57613dc Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T07:45:53Z small fix commit c31562642eaf2fa9bd35366246430fa6de8be6d8 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:46:43Z Revert "small fix" This reverts commit 67d1b21661886b204bbb86bc472a2ecce57613dc. commit ef1d9a8921d2e5060824878d7212b1d50454e0f9 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:48:44Z Revert "fix" This reverts commit 320386d2c484dac7eed9b7fe1584ca376e4ad897. commit 5c09181de11a5cdae78acf690513f0166a1d5c21 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-09T10:48:58Z Revert "KAFKA-2944: fix NullPointerException in KafkaConfigStorage" This reverts commit 82150dccc59ac0e436acc5186d2b8fb66c9df671. commit eba2a0737ae93f3d8e0137c5901277f53c62daac Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T08:25:36Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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: Trunk kafka 3054
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/782 Trunk kafka 3054 The exceptions may propagate to DistributedHerder when start or stop connectors or tasks; The best solution is to track the track the status of connectors and tasks; This patch catch exceptions to guard the DistributedHerder; this a short-term solution; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-3054 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/782.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 #782 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit e75c8bbb3329868f92becc86645d31fa21c6c7f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-17T03:11:45Z KAFKA-3054: Connect Herder fail if sent a wrong connector config or task config, catch exceptions to guard --- 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-3054: Connect Herder fail if sent a wron...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/782 KAFKA-3054: Connect Herder fail if sent a wrong connector config or task config, catch exceptions to guard The exceptions may propagate to DistributedHerder when start or stop connectors or tasks; The best solution is to track the track the status of connectors and tasks; This patch catch exceptions to guard the DistributedHerder; this a short-term solution; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-3054 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/782.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 #782 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit e75c8bbb3329868f92becc86645d31fa21c6c7f4 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-17T03:11:45Z KAFKA-3054: Connect Herder fail if sent a wrong connector config or task config, catch exceptions to guard --- 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-3054: Connect Herder fail if sent a wron...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/782 --- 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-2934: Offset storage file configuration ...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/734 --- 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-2934: Offset storage file configuration ...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/734 KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig Added offsetBackingStore config to StandaloneConfig and DistributedConfig; Added config for offset.storage.topic and config.storage.topic into DistributedConfig; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2934 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/734.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 #734 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit dd90fb07a073de2f58ba6d868348aeaf9e70a6a3 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-02T12:49:18Z KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig commit ff572f805de3eb7de94046f6e29aee7254262d4c Author: jinxing <jinx...@fenbi.com> Date: 2016-01-02T14:21:19Z fix commit 394a1ab9e757d70d2cb83c70afcfbc5c13ca08a1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T08:43:58Z Revert "fix" This reverts commit ff572f805de3eb7de94046f6e29aee7254262d4c. commit 1eff43becd670add7e437dec3e2f34cb384b4ccf Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T08:44:10Z Revert "KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig" This reverts commit dd90fb07a073de2f58ba6d868348aeaf9e70a6a3. commit 133cb430c091957dc721bc9295dccbb98774b24c Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T09:09:21Z KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/693 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit d56be0b9e0849660c07d656c6019f9cc2f17ae55 Author: ZoneMayor <jinxing6...@126.com> Date: 2016-01-10T09:24:06Z Merge pull request #14 from apache/trunk 2016-1-10 commit a937ad38ac90b90a57a1969bdd8ce06d6faaaeb1 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T10:28:18Z Merge branch 'trunk-KAFKA-2875' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2875 commit 83b2bcca237ba9445360bbfcb05a0de82c36274f Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T12:39:20Z KAFKA-2875: wip commit 6e6f2c20c5730253d8e818c2dc1e5e741a05ac08 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-10T14:53:28Z KAFKA-2875: Classpath contains multiple SLF4J bindings warnings when using scripts under bin --- 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-2934: Offset storage file configuration ...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/734 KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig Added offsetBackingStore config to StandaloneConfig and DistributedConfig; Added config for offset.storage.topic and config.storage.topic into DistributedConfig; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2934 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/734.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 #734 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit dd90fb07a073de2f58ba6d868348aeaf9e70a6a3 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-02T12:49:18Z KAFKA-2934: Offset storage file configuration in Connect standalone mode is not included in StandaloneConfig commit ff572f805de3eb7de94046f6e29aee7254262d4c Author: jinxing <jinx...@fenbi.com> Date: 2016-01-02T14:21:19Z fix --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage commit 320386d2c484dac7eed9b7fe1584ca376e4ad897 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T06:09:00Z fix commit 67d1b21661886b204bbb86bc472a2ecce57613dc Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T07:45:53Z small fix --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage commit 320386d2c484dac7eed9b7fe1584ca376e4ad897 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T06:09:00Z fix commit 67d1b21661886b204bbb86bc472a2ecce57613dc Author: jinxing <jinx...@fenbi.com> Date: 2016-01-03T07:45:53Z small fix --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/723 --- 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-2944: fix NullPointerException in KafkaC...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/723 KAFKA-2944: fix NullPointerException in KafkaConfigStorage Lost of "config messages" can affect the logic of KafkaConfigStorage; Call readToEnd after sending each message to KafkaBasedLog to ensure that all config messages are flushed to Kafka; Since "config messages" sending to KafkaBasedLog are metadata, it will not affect performance too much; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2944 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/723.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 #723 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 52d02f333e86d06cfa8fff5facd18999b3db6d83 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-30T03:08:08Z Merge pull request #13 from apache/trunk 2015-12-30 commit 82150dccc59ac0e436acc5186d2b8fb66c9df671 Author: jinxing <jinx...@fenbi.com> Date: 2016-01-01T15:21:41Z KAFKA-2944: fix NullPointerException in KafkaConfigStorage --- 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-2977: Transient Failure in kafka.log.Log...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/671 --- 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-2977: Transient Failure in kafka.log.Log...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/671 KAFKA-2977: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest Code is as below: val appends = writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) cleaner.startup() val firstDirty = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty) val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) val firstDirty2 = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty2) log cleaner and writeDups are two different threads; log cleaner do cleaning every 15s, timeout in "cleaner.awaitCleaned" is 60s; there is a filtering condition for a log to be chosen to become a cleaning target: cleanableRatio> 0.5(configured log.cleaner.min.cleanable.ratio) by default; It may happen that, during "val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec))", log is also undergoing a cleaning process; Since the segment size configured in this test is quite small: 100, there is possibility that before the end of 'writeDups', some 'dirty segment' of the log is already cleaned; With tiny dirty part left, cleanableRatio> 0.5 cannot be satisfied; thus firstDirty2>lastCleaned2, which leads to this test failed; To fix, make MinCleanableDirtyRatioProp configurable in makeCleaner; Also removed minDirtyMessages. You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2977 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/671.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 #671 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit 0070c2d71d06ee8baa1cddb3451cd5af6c6b1d4a Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-11T14:50:30Z Merge pull request #8 from apache/trunk 2015-12-11 commit 09908ac646d4c84f854dad63b8c99213b74a7063 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-13T14:17:19Z Merge pull request #9 from apache/trunk 2015-12-13 commit ff1e68bb7101d12624c189174ef1dceb21ed9798 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:31:34Z KAFKA-2054: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest commit 6321ab6599cb7a981fac2a4eea64a5f2ea805dd6 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:36:11Z removed unnecessary maven repo commit 05cae52c72a02c0ed40fd4e3be03e1cb19f33f7a Author: jinxing <jinx...@fenbi.com> Date: 2015-12-17T12:21:12Z removed the semicolon commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 651de48663cf375ea714cdbeb34650d75f1f4d71 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:43:38Z KAFKA-2977: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest commit ba03eee18045dc4aabc56ff17907036c238b1f7d Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:50:57Z KAFKA-2977: fix commit 341c0f7d9d8a6910bc81a1cc9da6608c48323201 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T17:08:43Z Merge branch 'trunk-KAFKA-2977' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2977 commit 734d1d5ee179cf8671426ec770c8cab5f6228112 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T17:23:15Z KAFKA-2977: using String interpolation commit 55430ed040354279f06bd59ff1a62c2a50c8882c Author: jinxing <jinx...@fenbi.com> Date: 2015-12-19T13:39:39Z make MinCleanableDirtyRatioProp configurable in makeCleaner and removed minDirtyMessages commit 37d3e41f120a16da5571c9082147efb422cfc2d9 Author: jinxing <jinx...@fen
[GitHub] kafka pull request: KAFKA-2455: Test Failure: kafka.consumer.Metri...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/694 KAFKA-2455: Test Failure: kafka.consumer.MetricsTest > testMetricsLeak @guozhangwang I reproduced this; Every time this test failed, I found extra "type=DelayedFetchMetrics, name=ExpiresperSec, fetcherType=consumer/follower" inside Metrics.defaultRegistry().allMetrics().keySet(); that is because for "DelayedFetch", object DelayedFetchMetrics are loaded dynamically in function "onExpiration"; To fix, I explicitly load this object; Is it appropriate? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2455 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/694.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 #694 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit 5e6284e034fc8e92745f1ebdda2fa228b8beaf4f Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T11:17:17Z KAFKA-2455: Test Failure: kafka.consumer.MetricsTest > testMetricsLeak --- 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-2977: Transient Failure in kafka.log.Log...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/671 --- 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-2977: Transient Failure in kafka.log.Log...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/671 KAFKA-2977: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest Hi @guozhangwang Code is as below: val appends = writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) cleaner.startup() val firstDirty = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty) val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) val firstDirty2 = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty2) log cleaner and writeDups are two different threads; log cleaner do cleaning every 15s, timeout in "cleaner.awaitCleaned" is 60s; there is a filtering condition for a log to be chosen to become a cleaning target: cleanableRatio> 0.5(configured log.cleaner.min.cleanable.ratio) by default; It may happen that, during "val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec))", log is also undergoing a cleaning process; Since the segment size configured in this test is quite small: 100, there is possibility that before the end of 'writeDups', some 'dirty segment' of the log is already cleaned; With tiny dirty part left, cleanableRatio> 0.5 cannot be satisfied; thus firstDirty2>lastCleaned2, which leads to this test failed; Does it make sense? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2977 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/671.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 #671 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit 0070c2d71d06ee8baa1cddb3451cd5af6c6b1d4a Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-11T14:50:30Z Merge pull request #8 from apache/trunk 2015-12-11 commit 09908ac646d4c84f854dad63b8c99213b74a7063 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-13T14:17:19Z Merge pull request #9 from apache/trunk 2015-12-13 commit ff1e68bb7101d12624c189174ef1dceb21ed9798 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:31:34Z KAFKA-2054: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest commit 6321ab6599cb7a981fac2a4eea64a5f2ea805dd6 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:36:11Z removed unnecessary maven repo commit 05cae52c72a02c0ed40fd4e3be03e1cb19f33f7a Author: jinxing <jinx...@fenbi.com> Date: 2015-12-17T12:21:12Z removed the semicolon commit 651de48663cf375ea714cdbeb34650d75f1f4d71 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:43:38Z KAFKA-2977: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest commit ba03eee18045dc4aabc56ff17907036c238b1f7d Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:50:57Z KAFKA-2977: fix --- 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-2058: ProducerTest.testSendWithDeadBroke...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/689 KAFKA-2058: ProducerTest.testSendWithDeadBroker transient failure I reproduced this transient failure; It turns that waitUntilMetadataIsPropagated is not enough; in "onBrokerStartup", methods below will send send both LeaderAndIsrRequest and UpdateMetadataRequest to KafkaApis: replicaStateMachine.handleStateChanges(allReplicasOnNewBrokers, OnlineReplica) partitionStateMachine.triggerOnlinePartitionStateChange() The two kinds of request are handled seperately and we are not sure about the order; If UpdateMetadataRequest is handled first, metadataCache of kafkaApis will be updated, thus TestUtils.waitUntilMetadataIsPropagated will be satisfied, and consumer can(will) start fetching data; But if the LeaderAndIsrRequest is not handled at this moment, "becomeLeaderOrFollower" cannot be called , thus structures like "leaderReplicaOpt" cannot be updated, which leads to failure of consumer's fetching data; To fix above, consumer should start fetching data after partition's leaderReplica is refreshed, not just the leader is elected; You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2058 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/689.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 #689 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit 0070c2d71d06ee8baa1cddb3451cd5af6c6b1d4a Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-11T14:50:30Z Merge pull request #8 from apache/trunk 2015-12-11 commit 09908ac646d4c84f854dad63b8c99213b74a7063 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-13T14:17:19Z Merge pull request #9 from apache/trunk 2015-12-13 commit 30b26b2d3c714bff11f4c58f00f5d1b075a592e9 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-17T12:27:27Z Merge pull request #10 from apache/trunk 2015-12-17 commit 6b1790b2742fa1244d3ba44aef459d8d5a6d3b55 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-17T12:30:38Z KAFKA-2058: 30b26b2d3c714bff11f4c58f00f5d1b075a592e9 --- 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-2875: Class path contains multiple SLF4J...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/669 --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/693 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution hi @ijuma I reopened this pr again (sorry for my inexperience using github); I think I did much deduplication for the script; Please have a look when you have time : - ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/693.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 #693 commit 34240b52e1b70aa172b65155f6042243d838b420 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-18T07:22:20Z Merge pull request #12 from apache/trunk 2015-12-18 commit ffedf6fd04280e89978531fd73e7fe37a4d9bbed Author: jinxing <jinx...@fenbi.com> Date: 2015-12-18T07:24:14Z KAFKA-2875 Class path contains multiple SLF4J bindings warnings when using scripts under bin --- 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-2977: Transient Failure in kafka.log.Log...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/671 KAFKA-2977: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest Hi @guozhangwang - Code is as below: val appends = writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) cleaner.startup() val firstDirty = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty) val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec)) val firstDirty2 = log.activeSegment.baseOffset cleaner.awaitCleaned("log", 0, firstDirty2) -- log cleaner and writeDups are two different threads; log cleaner do cleaning every 15s, timeout in "cleaner.awaitCleaned" is 60s; there is a filtering condition for a log to be chosen to become a cleaning target: cleanableRatio> 0.5(configured log.cleaner.min.cleanable.ratio) by default; It may happen that, during "val appends2 = appends ++ writeDups(numKeys = 100, numDups = 3, log, CompressionCodec.getCompressionCodec(compressionCodec))", log is also undergoing a cleaning process; Since the segment size configured in this test is quite small: 100, there is possibility that before the end of 'writeDups', some 'dirty segment' of the log is already cleaned; With tiny dirty part left, cleanableRatio> 0.5 cannot be satisfied; thus firstDirty2>lastCleaned2, which leads to this test failed; Does it make sense? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2977 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/671.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 #671 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit 0070c2d71d06ee8baa1cddb3451cd5af6c6b1d4a Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-11T14:50:30Z Merge pull request #8 from apache/trunk 2015-12-11 commit 09908ac646d4c84f854dad63b8c99213b74a7063 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-13T14:17:19Z Merge pull request #9 from apache/trunk 2015-12-13 commit ff1e68bb7101d12624c189174ef1dceb21ed9798 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:31:34Z KAFKA-2054: Transient Failure in kafka.log.LogCleanerIntegrationTest.cleanerTest commit 6321ab6599cb7a981fac2a4eea64a5f2ea805dd6 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-13T14:36:11Z removed unnecessary maven repo --- 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-2875: remove slf4j multi binding warning...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/595 --- 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-2875:
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/669 KAFKA-2875: Hi @ijuma I repopened this PR; I make slf4jlog4j12 dependency version to be 1.7.13, by this way, the 1.7.6 transitive dependency version will be overriten I think; From my point of view, a proper way to fix the multi binding of slf4j is to specify the classpath inside of parent script of kafka-run-class.sh, like kafka-topics.sh, kafka-console-consumer.sh; As a result, kafka-run-class.sh will be told which command to run and where to find jars contains the class for the command; Am I right? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/669.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 #669 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit 0070c2d71d06ee8baa1cddb3451cd5af6c6b1d4a Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-11T14:50:30Z Merge pull request #8 from apache/trunk 2015-12-11 commit fe1f7fdb73c75dea5247bc5c4c9e78fdad6fea37 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-11T14:52:24Z KAFKA-2875: Class path contains multiple SLF4J bindings warnings when using scripts under bin commit 5047602438c2e6d66514ebfed07002404cc01578 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-11T14:54:43Z KAFKA-2875: WIP --- 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-2837: fix transient failure of kafka.api...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/648 --- 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-2837: fix transient failure of kafka.api...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/648 KAFKA-2837: fix transient failure of kafka.api.ProducerBounceTest > testBrokerFailure I can reproduced this transient failure, it seldom happen; code is like below: // rolling bounce brokers for (i <- 0 until numServers) { for (server <- servers) { server.shutdown() server.awaitShutdown() server.startup() Thread.sleep(2000) } // Make sure the producer do not see any exception // in returned metadata due to broker failures assertTrue(scheduler.failed == false) // Make sure the leader still exists after bouncing brokers (0 until numPartitions).foreach(partition => TestUtils.waitUntilLeaderIsElectedOrChanged(zkUtils, topic1, partition)) Brokers keep rolling restart, and producer keep sending messages; In every loop, it will wait for election of partition leader; But if the election is slow, more messages will be buffered in RecordAccumulator's BufferPool; The limit for buffer is set to be 3; TimeoutException("Failed to allocate memory within the configured max blocking time") will show up when out of memory; Since for every restart of the broker, it will sleep for 2000 ms, so this transient failure seldom happen; But if I reduce the sleeping period, the bigger chance failure happens; for example if the broker with role of controller suffered a restart, it will take time to select controller first, then select leader, which will lead to more messges blocked in KafkaProducer:RecordAccumulator:BufferPool; In this fix, I just enlarge the producer's buffer size to be 1MB; @guozhangwang , Could you give some comments? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2837 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/648.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 #648 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit cd5e6f4700a4387f9383b84aca0ee9c4639b1033 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-09T13:49:07Z KAFKA-2837: fix transient failure kafka.api.ProducerBounceTest > testBrokerFailure commit 8ded9104a04861f789a7a990c2ddd4fc38a899cd Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-10T04:47:06Z Merge pull request #6 from apache/trunk 2015-12-10 commit 2bcf010c73923bb24bbd9cece7e39983b2bdce0c Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:47:39Z KAFKA-2837: WIP commit dae4a3cc0b564bb25121d54e65b5ad363c3e866d Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:48:21Z Merge branch 'trunk-KAFKA-2837' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2837 commit 7118e11813e445bca3eab65a23028e76138b136a Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:51:43Z KAFKA-2837: WIP commit 310dd6b34547b52aad21a35dcf631bda3e15ab64 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-11T03:43:32Z KAFKA-2837: WIP --- 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-2837: fix transient failure of kafka.api...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/648 KAFKA-2837: fix transient failure of kafka.api.ProducerBounceTest > testBrokerFailure I can reproduced this transient failure, it seldom happen; code is like below: // rolling bounce brokers for (i <- 0 until numServers) { for (server <- servers) { server.shutdown() server.awaitShutdown() server.startup() Thread.sleep(2000) } // Make sure the producer do not see any exception // in returned metadata due to broker failures assertTrue(scheduler.failed == false) // Make sure the leader still exists after bouncing brokers (0 until numPartitions).foreach(partition => TestUtils.waitUntilLeaderIsElectedOrChanged(zkUtils, topic1, partition)) Brokers keep rolling restart, and producer keep sending messages; In every loop, it will wait for election of partition leader; But if the election is slow, more messages will be buffered in RecordAccumulator's BufferPool; The limit for buffer is set to be 3; TimeoutException("Failed to allocate memory within the configured max blocking time") will show up when out of memory; Since for every restart of the broker, it will sleep for 2000 ms, so this transient failure seldom happen; But if I reduce the sleeping period, the bigger chance failure happens; for example if the broker with role of controller suffered a restart, it will take time to select controller first, then select leader, which will lead to more messges blocked in KafkaProducer:RecordAccumulator:BufferPool; In this fix, I just enlarge the producer's buffer size to be 1MB; @guozhangwang , Could you give some comments? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2837 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/648.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 #648 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit cd5e6f4700a4387f9383b84aca0ee9c4639b1033 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-09T13:49:07Z KAFKA-2837: fix transient failure kafka.api.ProducerBounceTest > testBrokerFailure commit 8ded9104a04861f789a7a990c2ddd4fc38a899cd Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-10T04:47:06Z Merge pull request #6 from apache/trunk 2015-12-10 commit 2bcf010c73923bb24bbd9cece7e39983b2bdce0c Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:47:39Z KAFKA-2837: WIP commit dae4a3cc0b564bb25121d54e65b5ad363c3e866d Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:48:21Z Merge branch 'trunk-KAFKA-2837' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2837 commit 7118e11813e445bca3eab65a23028e76138b136a Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:51:43Z KAFKA-2837: WIP --- 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-2837: fix transient failure of kafka.api...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/648 --- 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-2837: fix transient failure of kafka.api...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/648 --- 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-2837: fix transient failure of kafka.api...
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/648 KAFKA-2837: fix transient failure of kafka.api.ProducerBounceTest > testBrokerFailure I can reproduced this transient failure, it seldom happen; code is like below: // rolling bounce brokers for (i <- 0 until numServers) { for (server <- servers) { server.shutdown() server.awaitShutdown() server.startup() Thread.sleep(2000) } // Make sure the producer do not see any exception // in returned metadata due to broker failures assertTrue(scheduler.failed == false) // Make sure the leader still exists after bouncing brokers (0 until numPartitions).foreach(partition => TestUtils.waitUntilLeaderIsElectedOrChanged(zkUtils, topic1, partition)) Brokers keep rolling restart, and producer keep sending messages; In every loop, it will wait for election of partition leader; But if the election is slow, more messages will be buffered in RecordAccumulator's BufferPool; The limit for buffer is set to be 3; TimeoutException("Failed to allocate memory within the configured max blocking time") will show up when out of memory; Since for every restart of the broker, it will sleep for 2000 ms, so this transient failure seldom happen; But if I reduce the sleeping period, the bigger chance failure happens; for example if the broker with role of controller suffered a restart, it will take time to select controller first, then select leader, which will lead to more messges blocked in KafkaProducer:RecordAccumulator:BufferPool; In this fix, I just enlarge the producer's buffer size to be 1MB; @guozhangwang , Could you give some comments? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2837 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/648.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 #648 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit cd5e6f4700a4387f9383b84aca0ee9c4639b1033 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-09T13:49:07Z KAFKA-2837: fix transient failure kafka.api.ProducerBounceTest > testBrokerFailure commit 8ded9104a04861f789a7a990c2ddd4fc38a899cd Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-10T04:47:06Z Merge pull request #6 from apache/trunk 2015-12-10 commit 2bcf010c73923bb24bbd9cece7e39983b2bdce0c Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:47:39Z KAFKA-2837: WIP commit dae4a3cc0b564bb25121d54e65b5ad363c3e866d Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:48:21Z Merge branch 'trunk-KAFKA-2837' of https://github.com/ZoneMayor/kafka into trunk-KAFKA-2837 commit 7118e11813e445bca3eab65a23028e76138b136a Author: jinxing <jinx...@fenbi.com> Date: 2015-12-10T04:51:43Z KAFKA-2837: WIP --- 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: Trunk kafka 2839
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/602 --- 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-2837: fix transient failure of kafka.api...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/648 KAFKA-2837: fix transient failure of kafka.api.ProducerBounceTest > testBrokerFailure I can reproduced this transient failure, it seldom happen; code is like below: // rolling bounce brokers for (i <- 0 until numServers) { for (server <- servers) { server.shutdown() server.awaitShutdown() server.startup() Thread.sleep(2000) } // Make sure the producer do not see any exception // in returned metadata due to broker failures assertTrue(scheduler.failed == false) // Make sure the leader still exists after bouncing brokers (0 until numPartitions).foreach(partition => TestUtils.waitUntilLeaderIsElectedOrChanged(zkUtils, topic1, partition)) Brokers keep rolling restart, and producer keep sending messages; In every loop, it will wait for election of partition leader; But if the election is slow, more messages will be buffered in RecordAccumulator's BufferPool; The limit for buffer is set to be 3; TimeoutException("Failed to allocate memory within the configured max blocking time") will show up when out of memory; Since for every restart of the broker, it will sleep for 2000 ms, so this transient failure seldom happen; But if I reduce the sleeping period, the bigger chance failure happens; for example if the broker with role of controller suffered a restart, it will take time to select controller first, then select leader, which will lead to more messges blocked in KafkaProducer:RecordAccumulator:BufferPool; In this fix, I just enlarge the producer's buffer size to be 1MB; @guozhangwang , Could you give some comments? You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2837 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/648.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 #648 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit b767a8dff85fc71c75d4cf5178c3f6f03ff81bfc Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-09T10:42:30Z Merge pull request #5 from apache/trunk 2015-12-9 commit cd5e6f4700a4387f9383b84aca0ee9c4639b1033 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-09T13:49:07Z KAFKA-2837: fix transient failure kafka.api.ProducerBounceTest > testBrokerFailure --- 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-2893: Add a simple non-negative partitio...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/628 KAFKA-2893: Add a simple non-negative partition seek check You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2893 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/628.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 #628 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit cec5b48b651a7efd3900cfa3c1fd0ab1eeeaa3ec Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-01T10:44:02Z Merge pull request #2 from apache/trunk 2015-12-1 commit a119d547bf1741625ce0627073c7909992a20f15 Author: ZoneMayor <jinxing6...@126.com> Date: 2015-12-04T13:42:27Z Merge pull request #3 from apache/trunk 2015-12-04#KAFKA-2893 commit 253769e9441bc0634fd27d00375c5381daf03202 Author: jinxing <jinx...@fenbi.com> Date: 2015-12-04T14:01:42Z KAFKA-2893: Add Negative Partition Seek Check --- 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: Trunk kafka 2839
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/602 Trunk kafka 2839 I believe there is a bug in KafkaBasedLogTest implementation; Records are appended to "KafkaBasedLog" by : consumer.addRecord(new ConsumerRecord<>(TOPIC, 0, 0, TP0_KEY, TP0_VALUE)); consumer.addRecord(new ConsumerRecord<>(TOPIC, 0, 1, TP0_KEY, TP0_VALUE_NEW)); consumer.addRecord(new ConsumerRecord<>(TOPIC, 1, 0, TP1_KEY, TP1_VALUE)); consumer.addRecord(new ConsumerRecord<>(TOPIC, 1, 1, TP1_KEY, TP1_VALUE_NEW)); and achieved by "consumedRecords"; Inside of "ConsumerRecords", in order to turn "records", which is a HashMap<TopicPartition, List<ConsumerRecord<K, V>>>, into Iterator<ConsumerRecord<K, V>>; We can just guarantee the sequence underneath same key, while we cannot assure the sequence under different keys; Am I wrong? @gwenshap You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2839 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/602.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 #602 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit 0875176f90dbece21e1777902a079cf67ad157a7 Author: jinxing <jinx...@fenbi.com> Date: 2015-11-30T03:39:29Z KAFKA-2839: fix KafkaBasedLogTest testSendAndReadToEnd failing --- 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-2875: remove slf4j multi binding warning...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/595 KAFKA-2875: remove slf4j multi binding warnings when running form source distribution You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk-KAFKA-2875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/595.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 #595 commit 95374147a28208d4850f6e73f714bf418935fc2d Author: ZoneMayor <jinxing6...@126.com> Date: 2015-11-27T03:49:34Z Merge pull request #1 from apache/trunk merge commit 58bb01c47a1054301d7416503d51912611473087 Author: jinxing <jinx...@fenbi.com> Date: 2015-11-27T08:15:57Z KAFKA-2875: remove slf4j multi binding warnings when running form source distribution --- 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-2361: unit test failure in ProducerFailu...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/571 --- 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-2361: unit test failure in ProducerFailu...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/571 KAFKA-2361: unit test failure in ProducerFailureHandlingTest.testNotE⦠same issue with KAFKA-1999, so I want to fix it https://issues.apache.org/jira/browse/KAFKA-1999 You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka 0.8.2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/571.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 #571 commit da1d0d88cd4fd8819b2bbc1068a4473062a041e9 Author: jinxing <jinx...@fenbi.com> Date: 2015-11-19T03:42:05Z KAFKA-2361: unit test failure in ProducerFailureHandlingTest.testNotEnoughReplicasAfterBrokerShutdown --- 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-2361: unit test failure in ProducerFailu...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/558 --- 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-2361: unit test failure in ProducerFailu...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/558 KAFKA-2361: unit test failure in ProducerFailureHandlingTest.testNotE⦠the same issue stated at https://issues.apache.org/jira/browse/KAFKA-1999, but on branch 0.8.2, still not fixed @junrao You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka 0.8.2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/558.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 #558 commit da1d0d88cd4fd8819b2bbc1068a4473062a041e9 Author: jinxing <jinx...@fenbi.com> Date: 2015-11-19T03:42:05Z KAFKA-2361: unit test failure in ProducerFailureHandlingTest.testNotEnoughReplicasAfterBrokerShutdown --- 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-2605: Replace 'catch: Throwable' clauses...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/531 --- 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-2605: Replace 'catch: Throwable' clauses...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/532 KAFKA-2605: Replace 'catch: Throwable' clauses with 'NonFatal' 'catch: Throwable' will catch VirtualMachineError, ThreadDeath, InterruptedException, LinkageError, ControlThrowable, NotImplementedError; we don't want to catch those kind of error You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka 0.8.2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/532.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 #532 commit c466ef240e509c9aed7b2e7c6729df97abe89fff Author: jinxing <jinxing6...@126.com> Date: 2015-11-15T07:11:13Z KAFKA-2605: Replace 'catch: Throwable' clauses with 'NonFatal' --- 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-2605: Replace 'catch: Throwable' clauses...
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/532 --- 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-2605: Replace 'catch: Throwable' clauses...
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/531 KAFKA-2605: Replace 'catch: Throwable' clauses with 'NonFatal' 'catch: Throwable' will catch VirtualMachineError, ThreadDeath, InterruptedException, LinkageError, ControlThrowable, NotImplementedError; we don't want to catch those kind of error You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka 0.8.2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/531.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 #531 commit c466ef240e509c9aed7b2e7c6729df97abe89fff Author: jinxing <jinxing6...@126.com> Date: 2015-11-15T07:11:13Z KAFKA-2605: Replace 'catch: Throwable' clauses with 'NonFatal' --- 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: more test
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/228 --- 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: more test
GitHub user ZoneMayor reopened a pull request: https://github.com/apache/kafka/pull/228 more test You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/228.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 #228 commit dc23d0a0d724cefd550e95fd7dc58230cf480d0b Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:25:37Z MOD: for test commit 87ed0b5e71991e756aaeb2d8012b89b1c80b38d1 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:42:39Z MOD: another for test commit c13d6a5e1920cd5c11e96c524527dfb597f5adc4 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:48:09Z MOD: okok-test commit 2faa61b10965d9b6e7990158c8e492c711a16fb5 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:59:21Z MOD: more test --- 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: more test
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/228 --- 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: MOD: for test
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/225 MOD: for test You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/225.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 #225 commit dc23d0a0d724cefd550e95fd7dc58230cf480d0b Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:25:37Z MOD: for test --- 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: MOD: for test
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/225 --- 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: MOD: test again
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/226 MOD: test again You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/226.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 #226 commit dc23d0a0d724cefd550e95fd7dc58230cf480d0b Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:25:37Z MOD: for test commit 87ed0b5e71991e756aaeb2d8012b89b1c80b38d1 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:42:39Z MOD: another for test --- 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: MOD: test again
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/226 --- 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: okok-test
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/227 okok-test You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/227.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 #227 commit dc23d0a0d724cefd550e95fd7dc58230cf480d0b Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:25:37Z MOD: for test commit 87ed0b5e71991e756aaeb2d8012b89b1c80b38d1 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:42:39Z MOD: another for test commit c13d6a5e1920cd5c11e96c524527dfb597f5adc4 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:48:09Z MOD: okok-test --- 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: okok-test
Github user ZoneMayor closed the pull request at: https://github.com/apache/kafka/pull/227 --- 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: more test
GitHub user ZoneMayor opened a pull request: https://github.com/apache/kafka/pull/228 more test You can merge this pull request into a Git repository by running: $ git pull https://github.com/ZoneMayor/kafka trunk Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/228.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 #228 commit dc23d0a0d724cefd550e95fd7dc58230cf480d0b Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:25:37Z MOD: for test commit 87ed0b5e71991e756aaeb2d8012b89b1c80b38d1 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:42:39Z MOD: another for test commit c13d6a5e1920cd5c11e96c524527dfb597f5adc4 Author: jinxing <jinx...@fenbi.com> Date: 2015-09-21T03:48:09Z MOD: okok-test --- 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. ---