Author: hrsakai
Date: Tue Jan 30 07:21:49 2018
New Revision: 24542
Log:
Added gpg key for hrsakai
Modified:
release/incubator/pulsar/KEYS
Modified: release/incubator/pulsar/KEYS
==
--- release/incubator/pulsar/KEYS
Author: hrsakai
Date: Tue Jan 30 07:15:32 2018
New Revision: 24541
Log:
Added gpg key for hrsakai
Modified:
dev/incubator/pulsar/KEYS
Modified: dev/incubator/pulsar/KEYS
==
--- dev/incubator/pulsar/KEYS (original)
jai1 commented on a change in pull request #1123: Configure a max
producer/consumer queue size enforced across partitions
URL: https://github.com/apache/incubator-pulsar/pull/1123#discussion_r164650547
##
File path: pulsar-client-cpp/lib/ProducerConfiguration.cc
##
@@
hrsakai opened a new pull request #1146: Fix url option on client tool
URL: https://github.com/apache/incubator-pulsar/pull/1146
### Motivation
We can't use `pulsar-client` command without specifing
`serviceUrl`,`brokerServiceUrl`, or `webServiceUrl`.
```
$ cat
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 059149c Fix race condition in
merlimat closed issue #814: Intermittent test failure
DispatcherBlockConsumerTest.testRedeliveryOnBlockedDistpatcher
URL: https://github.com/apache/incubator-pulsar/issues/814
This is an automated message from the Apache
merlimat commented on issue #1145: Intermittent test failure in
PersistentTopicTest.testClosingReplicationProducerTwice
URL:
https://github.com/apache/incubator-pulsar/issues/1145#issuecomment-361471241
This reproduces locally after running ~900 times.
merlimat opened a new issue #1145: Intermittent test failure in
PersistentTopicTest.testClosingReplicationProducerTwice
URL: https://github.com/apache/incubator-pulsar/issues/1145
As seen in
merlimat opened a new pull request #1144: Fix race condition in
testRedeliveryOnBlockedDistpatcher
URL: https://github.com/apache/incubator-pulsar/pull/1144
### Motivation
The test is using `receiveAsync()` over multiple consumer objects and adding
the message ids on an
merlimat commented on a change in pull request #1135: Added infinite time
retention configuration option
URL: https://github.com/apache/incubator-pulsar/pull/1135#discussion_r164624867
##
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java
merlimat commented on issue #1097: Validate expected types in Python code
wrapper
URL: https://github.com/apache/incubator-pulsar/pull/1097#issuecomment-361454629
retest this please
This is an automated message from the
merlimat closed pull request #1139: Add timeouts to setup & teardown for
ReplicatorTest
URL: https://github.com/apache/incubator-pulsar/pull/1139
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:
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 8a2b2f0 Add timeouts to setup &
rdhabalia commented on a change in pull request #1135: Added infinite time
retention configuration option
URL: https://github.com/apache/incubator-pulsar/pull/1135#discussion_r164621919
##
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java
rdhabalia commented on issue #1002: Making Pulsar Proxy more secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#issuecomment-361450629
retest this please
This is an automated message from the Apache Git Service.
rdhabalia commented on issue #1002: Making Pulsar Proxy more secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#issuecomment-361450629
retest this please
This is an automated message from the Apache Git Service.
rdhabalia commented on a change in pull request #1002: Making Pulsar Proxy more
secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#discussion_r164619782
##
File path:
pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java
##
@@
rdhabalia commented on a change in pull request #1002: Making Pulsar Proxy more
secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#discussion_r164620502
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationManager.java
merlimat commented on issue #1003: Intermittent test failure
DispatcherBlockConsumerTest.testConsumerBlockingWithUnAckedMessagesAtDispatcher
URL:
https://github.com/apache/incubator-pulsar/issues/1003#issuecomment-361438295
Still happening:
merlimat closed issue #1082: Intermittent test failures in
DispatcherBlockConsumerTest.testConsumerBlockingWithUnAckedMessagesAtDispatcher
URL: https://github.com/apache/incubator-pulsar/issues/1082
This is an automated
nkurihar commented on issue #1133: Property name in documentation is incorrect
URL: https://github.com/apache/incubator-pulsar/pull/1133#issuecomment-361429583
LGTM ?
This is an automated message from the Apache Git Service.
merlimat commented on issue #1115: In Kafka consumer wrapper, group multiple
messages when the app polls
URL: https://github.com/apache/incubator-pulsar/pull/1115#issuecomment-361425686
retest this please
This is an
merlimat commented on issue #154: Retention 0mb and BacklogQuota 0mb
URL:
https://github.com/apache/incubator-pulsar/issues/154#issuecomment-361416957
Added 2nd PR for docs changes:
https://github.com/apache/incubator-pulsar/pull/1141
merlimat opened a new pull request #1141: Some clarifications on Retention
documentation
URL: https://github.com/apache/incubator-pulsar/pull/1141
### Motivation
Tried to simplify here a couple of statements on Retention and added
references to infinite retention time as defined in
jai1 commented on a change in pull request #1129: Added end to end encryption
in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164591461
##
File path: pulsar-client-cpp/lib/ProducerImpl.cc
##
@@ -50,7 +50,9 @@
lucperkins opened a new pull request #1140: Remove master/slave terminology
from Pulsar (WIP)
URL: https://github.com/apache/incubator-pulsar/pull/1140
### Motivation
There is a broad consensus in our industry that "master/slave" terminology
is damaging, unnecessary, and frequently
jai1 commented on a change in pull request #1129: Added end to end encryption
in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164589275
##
File path: pulsar-client-cpp/lib/ConsumerConfigurationImpl.h
##
@@ -31,13 +31,18 @@ struct
jai1 commented on a change in pull request #1129: Added end to end encryption
in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164588488
##
File path: pulsar-client-cpp/lib/ConsumerConfiguration.cc
##
@@ -74,4 +74,30 @@ void
jai1 commented on a change in pull request #1129: Added end to end encryption
in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164587853
##
File path: pulsar-client-cpp/include/pulsar/ConsumerCryptoFailureAction.h
##
@@ -0,0 +1,41 @@
merlimat opened a new pull request #1139: Add timeouts to setup & teardown for
ReplicatorTest
URL: https://github.com/apache/incubator-pulsar/pull/1139
### Motivation
All the tests in `ReplicatorTest` already do have timeouts, though the build
keeps getting stuck on this class. The
merlimat opened a new issue #1138: Test Timeouts in ReplicatorTest
URL: https://github.com/apache/incubator-pulsar/issues/1138
I have been seeing a lot of CI builds timing out after 100 minutes, all were
blocked on the `ReplicatorTest`. For example:
merlimat commented on issue #1126: Intermittent test failure in
testSkipCorruptDataLedger
URL:
https://github.com/apache/incubator-pulsar/issues/1126#issuecomment-361408690
@rdhabalia I have seen this test failing several times in the last few days.
Can you take a look when you have the
jai1 commented on issue #1002: Making Pulsar Proxy more secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#issuecomment-361408368
retest this please
This is an automated message from the Apache Git Service.
To
merlimat commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164583151
##
File path: pom.xml
##
@@ -180,7 +200,7 @@ flexible messaging model and an intuitive client
mgodave commented on issue #1112: Broker schema registry (1/3)
URL: https://github.com/apache/incubator-pulsar/pull/1112#issuecomment-361399155
Replaced with https://github.com/apache/incubator-pulsar/pull/1137
This is an
mgodave opened a new pull request #1137: Schema registry
URL: https://github.com/apache/incubator-pulsar/pull/1137
### Motivation
Allow users of the system to store the structure and format of the data in a
topic
PIP:
mgodave closed pull request #1112: Broker schema registry (1/3)
URL: https://github.com/apache/incubator-pulsar/pull/1112
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:
As this is a foreign pull
ivankelly commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164565853
##
File path: pom.xml
##
@@ -741,7 +761,7 @@ flexible messaging model and an intuitive client
ivankelly commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164565510
##
File path: pom.xml
##
@@ -180,7 +200,7 @@ flexible messaging model and an intuitive client
merlimat commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164565191
##
File path: pom.xml
##
@@ -741,7 +761,7 @@ flexible messaging model and an intuitive client
merlimat commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164564840
##
File path: pom.xml
##
@@ -180,7 +200,7 @@ flexible messaging model and an intuitive client
ivankelly opened a new pull request #1136: Read compacted consumer flag
URL: https://github.com/apache/incubator-pulsar/pull/1136
When enabled, a consumer will read messages from a compacted topic
ledger if available. When disabled, the consumer will read from the
message backlog as
merlimat commented on issue #1133: Property name in documentation is incorrect
URL: https://github.com/apache/incubator-pulsar/pull/1133#issuecomment-361380688
> @merlimat we should probably get someone who speaks japanese to look at
that one
Yep, sure. @yush1ga @nkurihar @massakam
ivankelly commented on issue #1042: Update maven compiler plugin to newest
version
URL: https://github.com/apache/incubator-pulsar/pull/1042#issuecomment-361374120
retest this please
This is an automated message from the
ivankelly commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164548657
##
File path: pulsar-broker/pom.xml
##
@@ -119,6 +119,34 @@
test
+
merlimat closed pull request #1044: Compact algo
URL: https://github.com/apache/incubator-pulsar/pull/1044
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:
As this is a foreign pull request (from a
merlimat commented on issue #154: Retention 0mb and BacklogQuota 0mb
URL:
https://github.com/apache/incubator-pulsar/issues/154#issuecomment-361367391
Sorry @sschepens, I missed this issue back in the days. I just created #1135
and I'll create another PR to clarify in the documentation
ivankelly commented on issue #1044: Compact algo
URL: https://github.com/apache/incubator-pulsar/pull/1044#issuecomment-361366560
@merlimat CI is green now. I can push some more changes if this gets merged.
This is an
ivankelly commented on issue #1133: Property name in documentation is incorrect
URL: https://github.com/apache/incubator-pulsar/pull/1133#issuecomment-361366061
@merlimat we should probably get someone who speaks japanese to look at that
one
merlimat opened a new pull request #1135: Added infinite time retention
configuration option
URL: https://github.com/apache/incubator-pulsar/pull/1135
### Motivation
Currently, there is no direct way to configure an "infinite" retention time
on data published on Pulsar topics,
merlimat commented on issue #1122: Use Optional for returning least
loaded broker
URL: https://github.com/apache/incubator-pulsar/pull/1122#issuecomment-361353480
retest this please
This is an automated message from the
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164523814
##
File path: pulsar-client-cpp/lib/MessageCrypto.h
##
@@ -0,0 +1,95 @@
+/**
+ * Licensed to the
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164524553
##
File path: pulsar-client-cpp/lib/ProducerImpl.cc
##
@@ -262,12 +295,21 @@ void
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164522672
##
File path: pulsar-client-cpp/include/pulsar/CryptoKeyReader.h
##
@@ -0,0 +1,51 @@
+/**
+ *
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164524311
##
File path: pulsar-client-cpp/lib/ProducerImpl.cc
##
@@ -262,12 +295,21 @@ void
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164524955
##
File path: pulsar-client-cpp/run-unit-tests.sh
##
@@ -23,7 +23,9 @@
rm -rf ./pulsar-dist
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164523212
##
File path: pulsar-client-cpp/lib/BatchMessageContainer.cc
##
@@ -93,6 +93,10 @@ void
merlimat commented on a change in pull request #1129: Added end to end
encryption in C++ client
URL: https://github.com/apache/incubator-pulsar/pull/1129#discussion_r164523753
##
File path: pulsar-client-cpp/lib/MessageCrypto.h
##
@@ -0,0 +1,95 @@
+/**
+ * Licensed to the
merlimat commented on issue #1044: Compact algo
URL: https://github.com/apache/incubator-pulsar/pull/1044#issuecomment-360880464
retest this please
This is an automated message from the Apache Git Service.
To respond to the
merlimat commented on a change in pull request #1124: Support Pulsar proxy from
C++/Python client library
URL: https://github.com/apache/incubator-pulsar/pull/1124#discussion_r164510154
##
File path: pulsar-client-cpp/lib/Url.cc
##
@@ -88,6 +88,12 @@ const std::string&
merlimat commented on a change in pull request #1124: Support Pulsar proxy from
C++/Python client library
URL: https://github.com/apache/incubator-pulsar/pull/1124#discussion_r164509672
##
File path: pulsar-client-cpp/lib/ConnectionPool.cc
##
@@ -33,36 +33,37 @@
jai1 commented on issue #1125: Added missing C++ enum values
URL: https://github.com/apache/incubator-pulsar/pull/1125#issuecomment-361327943
retest this please
This is an automated message from the Apache Git Service.
To
merlimat commented on a change in pull request #1044: Compact algo
URL: https://github.com/apache/incubator-pulsar/pull/1044#discussion_r164508229
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java
##
@@ -0,0 +1,282 @@
+/**
+ *
jai1 commented on a change in pull request #1124: Support Pulsar proxy from
C++/Python client library
URL: https://github.com/apache/incubator-pulsar/pull/1124#discussion_r164505677
##
File path: pulsar-client-cpp/lib/Url.h
##
@@ -41,6 +41,8 @@ class Url {
const
merlimat commented on issue #1047: Add StatefulSet option for BookKeeper on
Google Kubernetes Engine
URL: https://github.com/apache/incubator-pulsar/pull/1047#issuecomment-361325066
retest this please
This is an automated
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new b3fe055 Comment in config breaks
merlimat closed pull request #1131: Comment in config breaks docker scripts
URL: https://github.com/apache/incubator-pulsar/pull/1131
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:
As this is a
merlimat commented on issue #1134: [WIP] Docker compose files for pulsar
URL: https://github.com/apache/incubator-pulsar/pull/1134#issuecomment-361319778
Even though this is not technically a "prod" deployment, should we have it
under the `deployment` folder?
merlimat commented on a change in pull request #1132: Explicitly set zookeeper
version in broker
URL: https://github.com/apache/incubator-pulsar/pull/1132#discussion_r164501056
##
File path: pulsar-broker/pom.xml
##
@@ -119,6 +119,34 @@
test
+
Review
ivankelly commented on issue #1134: [WIP] Docker compose files for pulsar
URL: https://github.com/apache/incubator-pulsar/pull/1134#issuecomment-361294703
Currently only works with docker images generated with #1131 and #1132
applied.
ivankelly opened a new pull request #1134: Docker compose files for pulsar
URL: https://github.com/apache/incubator-pulsar/pull/1134
These files set up two clusters, US and EU, all on the local
machine. Each cluster has 2 pulsar brokers, 3 bookies, and a 2
zookeeper nodes, one in
ivankelly opened a new pull request #1133: Property name in documentation is
incorrect
URL: https://github.com/apache/incubator-pulsar/pull/1133
The setup guides specify that test-prop is the property name, but then
all later examples use test as the property name. This is obviously
ivankelly opened a new pull request #1132: Explicitly set zookeeper version in
broker
URL: https://github.com/apache/incubator-pulsar/pull/1132
The broker "shades" zookeeper via the aspectj stuff, but the version
it shades is the one pulled in with bookkeeper-server, not the version
ivankelly opened a new pull request #1131: Comment in config breaks docker
scripts
URL: https://github.com/apache/incubator-pulsar/pull/1131
docker/scripts/apply-config-from-env.py only accepts # comments, not
double slash.
jai1 commented on issue #1002: Making Pulsar Proxy more secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#issuecomment-361228242
retest this please
This is an automated message from the Apache Git Service.
To
75 matches
Mail list logo