[GitHub] afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153579930 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153579568 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153579421 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
afalko commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153580924 ## File path: ansible/deploy-pulsar.yaml ## @@ -0,0 +1,166 @@ +# +# Licensed to

[GitHub] merlimat closed pull request #919: Fix: broker-zookeeper latency-metrics's dimension-name

2017-11-28 Thread GitBox
merlimat closed pull request #919: Fix: broker-zookeeper latency-metrics's dimension-name URL: https://github.com/apache/incubator-pulsar/pull/919 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] lucperkins opened a new pull request #921: Restructure bare metal deployment documentation (WIP)

2017-11-28 Thread GitBox
lucperkins opened a new pull request #921: Restructure bare metal deployment documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/921 ### Motivation At the moment, the instructions on installing a bare metal cluster or instance are overly complex and in need of

[GitHub] merlimat commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
merlimat commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153646222 ## File path: pom.xml ## @@ -125,7 +125,7 @@ flexible messaging model and an intuitive client API.

[GitHub] lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153656217 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] merlimat commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
merlimat commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153646724 ## File path: build/docker/protobuf.patch ## @@ -161,30 +167,30 @@ index 4c087db..cd18e28 100644 "classname",

[GitHub] merlimat commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
merlimat commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153646828 ## File path: pulsar-common/src/test/java/org/apache/pulsar/common/api/DoubleByteBufTest.java ## @@ -50,5 +50,22 @@ public void

[GitHub] rdhabalia commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
rdhabalia commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153633906 ## File path: build/docker/protobuf.patch ## @@ -161,30 +167,30 @@ index 4c087db..cd18e28 100644 "classname",

[GitHub] rdhabalia commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
rdhabalia commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153639881 ## File path: pulsar-common/src/test/java/org/apache/pulsar/common/api/DoubleByteBufTest.java ## @@ -50,5 +50,22 @@ public void

[GitHub] rdhabalia commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
rdhabalia commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153638970 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/protobuf/ByteBufCodedOutputStream.java ## @@ -261,7

[GitHub] rdhabalia commented on a change in pull request #689: Netty 4.1

2017-11-28 Thread GitBox
rdhabalia commented on a change in pull request #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#discussion_r153636992 ## File path: pom.xml ## @@ -125,7 +125,7 @@ flexible messaging model and an intuitive client API.

[GitHub] lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153655770 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153655770 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable

[GitHub] merlimat commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

2017-11-28 Thread GitBox
merlimat commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option URL: https://github.com/apache/incubator-pulsar/pull/923#discussion_r153696154 ## File path: pom.xml ## @@ -106,15 +106,15 @@ flexible messaging model and an

[GitHub] merlimat closed pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

2017-11-28 Thread GitBox
merlimat closed pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option URL: https://github.com/apache/incubator-pulsar/pull/923 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rdhabalia commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

2017-11-28 Thread GitBox
rdhabalia commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option URL: https://github.com/apache/incubator-pulsar/pull/923#discussion_r153695650 ## File path: pom.xml ## @@ -106,15 +106,15 @@ flexible messaging model and

[GitHub] merlimat commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

2017-11-28 Thread GitBox
merlimat commented on a change in pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option URL: https://github.com/apache/incubator-pulsar/pull/923#discussion_r153696236 ## File path: pom.xml ## @@ -106,15 +106,15 @@ flexible messaging model and an

[GitHub] merlimat opened a new pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

2017-11-28 Thread GitBox
merlimat opened a new pull request #923: Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option URL: https://github.com/apache/incubator-pulsar/pull/923 ### Motivation Upgrade to bk-4.3.1.83-yahoo to expose journalSyncData option

[GitHub] merlimat closed issue #686: DuplicatedByteBuf is deprecated

2017-11-28 Thread GitBox
merlimat closed issue #686: DuplicatedByteBuf is deprecated URL: https://github.com/apache/incubator-pulsar/issues/686 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] jiazhai commented on issue #917: Provide an DCOS Universe package for pulsar

2017-11-28 Thread GitBox
jiazhai commented on issue #917: Provide an DCOS Universe package for pulsar URL: https://github.com/apache/incubator-pulsar/issues/917#issuecomment-347736962 Pulsar is composed of several apps (such as bookie, broker, monitor) together. But currently it only support to have one universe

[GitHub] jiazhai commented on issue #917: Provide an DCOS Universe package for pulsar

2017-11-28 Thread GitBox
jiazhai commented on issue #917: Provide an DCOS Universe package for pulsar URL: https://github.com/apache/incubator-pulsar/issues/917#issuecomment-347736962 Pulsar is composed of several apps (such as bookie, broker, monitor) together. But currently it only support to have one universe

[GitHub] zhaijack commented on issue #917: Provide an DCOS Universe package for pulsar

2017-11-28 Thread GitBox
zhaijack commented on issue #917: Provide an DCOS Universe package for pulsar URL: https://github.com/apache/incubator-pulsar/issues/917#issuecomment-347737386 Pulsar is composed of several apps (such as bookie, broker, monitor) together. But currently it only support to have one universe

[GitHub] zhaijack opened a new issue #924: Provide a json file for Marathon app groups to run Pulsar on DCOS

2017-11-28 Thread GitBox
zhaijack opened a new issue #924: Provide a json file for Marathon app groups to run Pulsar on DCOS URL: https://github.com/apache/incubator-pulsar/issues/924 There was a requirement to run Pulsar on DCOS in #917 , but Universe Package is not support well now. This ticket would like to

[GitHub] zhaijack commented on issue #917: Provide an DCOS Universe package for pulsar

2017-11-28 Thread GitBox
zhaijack commented on issue #917: Provide an DCOS Universe package for pulsar URL: https://github.com/apache/incubator-pulsar/issues/917#issuecomment-347738299 would like to left this issue tracking the DCOS supporting work, and open another issue #924 for the marathon app groups work.

[GitHub] merlimat commented on issue #689: Netty 4.1

2017-11-28 Thread GitBox
merlimat commented on issue #689: Netty 4.1 URL: https://github.com/apache/incubator-pulsar/pull/689#issuecomment-347722576 @rdhabalia Updated with comments This is an automated message from the Apache Git Service. To respond

[GitHub] sijie opened a new issue #922: Allow running bookie together with broker

2017-11-28 Thread GitBox
sijie opened a new issue #922: Allow running bookie together with broker URL: https://github.com/apache/incubator-pulsar/issues/922 Description: To simplify deployment, it will be good to run bookie along with broker. so people don't have to run separate bookkeeper cluster. It is

[GitHub] lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP)

2017-11-28 Thread GitBox
lucperkins commented on a change in pull request #920: Provide an Ansible playbook for AWS with documentation (WIP) URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153662966 ## File path: ansible/provision-pulsar-aws.tf ## @@ -0,0 +1,117 @@ +variable