rdhabalia commented on issue #2199: Cleanup unnecessary casting and creating
variables
URL: https://github.com/apache/incubator-pulsar/pull/2199#issuecomment-406092777
retest this please
This is an automated message from the
rdhabalia commented on issue #2198: Add support to configure subscription name
for sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2198#issuecomment-406092716
retest this please
This is an automated
rdhabalia removed a comment on issue #2198: Add support to configure
subscription name for sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2198#issuecomment-406092716
retest this please
This is an
rdhabalia removed a comment on issue #2199: Cleanup unnecessary casting and
creating variables
URL: https://github.com/apache/incubator-pulsar/pull/2199#issuecomment-406092777
retest this please
This is an automated message
merlimat closed pull request #2174: V2 doc changes
URL: https://github.com/apache/incubator-pulsar/pull/2174
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
merlimat commented on issue #2174: V2 doc changes
URL: https://github.com/apache/incubator-pulsar/pull/2174#issuecomment-406093286
Thanks @grantwwu !
This is an automated message from the Apache Git Service.
To respond to the
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 d8c0f10 V2 doc changes (#2174)
jiazhai commented on a change in pull request #2135: Allow user to get topic
from consumer message (#2007)
URL: https://github.com/apache/incubator-pulsar/pull/2135#discussion_r203549418
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java
merlimat commented on a change in pull request #2192: Rename Netty native libs
to reflect shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#discussion_r203549708
##
File path: src/rename-netty-native-libs.sh
##
@@ -0,0 +1,54 @@
+#!/bin/bash
+#
+#
merlimat commented on issue #2194: WIP: Destination fix
URL: https://github.com/apache/incubator-pulsar/pull/2194#issuecomment-406094383
@grantwwu merged #2174, you can rebase this on latest master
This is an automated
merlimat commented on issue #2197: unshade avro in pulsar-client-schema
URL: https://github.com/apache/incubator-pulsar/pull/2197#issuecomment-406094624
@jerrypeng There's a test failure:
```
Expected exception of type class
org.apache.pulsar.client.api.PulsarClientException but
rdhabalia commented on issue #2199: Cleanup unnecessary casting and creating
variables
URL: https://github.com/apache/incubator-pulsar/pull/2199#issuecomment-406110490
retest this please
This is an automated message from the
rdhabalia commented on a change in pull request #2192: Rename Netty native libs
to reflect shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#discussion_r203546380
##
File path: src/rename-netty-native-libs.sh
##
@@ -0,0 +1,54 @@
+#!/bin/bash
+#
+#
rdhabalia commented on a change in pull request #2192: Rename Netty native libs
to reflect shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#discussion_r203546130
##
File path: src/rename-netty-native-libs.sh
##
@@ -0,0 +1,54 @@
+#!/bin/bash
+#
+#
sijie closed issue #1001: Shading in Pulsar jars
URL: https://github.com/apache/incubator-pulsar/issues/1001
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
sijie closed issue #978: Documentation for compaction
URL: https://github.com/apache/incubator-pulsar/issues/978
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and
sijie commented on issue #978: Documentation for compaction
URL:
https://github.com/apache/incubator-pulsar/issues/978#issuecomment-406085036
Topic compaction documentation is available in
https://pulsar.incubator.apache.org/docs/latest/cookbooks/compaction/
sijie commented on issue #977: Rate limiting for compaction writes
URL:
https://github.com/apache/incubator-pulsar/issues/977#issuecomment-406085317
Close this now. Will recreate the issue if we need more comprehensive rate
limiting on compaction.
sijie closed issue #977: Rate limiting for compaction writes
URL: https://github.com/apache/incubator-pulsar/issues/977
This is an automated message from the Apache Git Service.
To respond to the message, please log on
sijie closed issue #983: optionally expose topic level metrics to prometheus
URL: https://github.com/apache/incubator-pulsar/issues/983
This is an automated message from the Apache Git Service.
To respond to the message,
ivankelly commented on issue #981: Cannot connect to a topic because of
ManagedLedgerException
URL:
https://github.com/apache/incubator-pulsar/issues/981#issuecomment-406084893
Do you have the full bookie log? It looks like the bookies are not
responding to read requests. Are all 3
rdhabalia opened a new pull request #2200: support subscription-type to be
passed in sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2200
### Motivation
Sometimes, user wants to maintain message-ordering while using
sink-function. So, sink-cli should have a
rdhabalia opened a new pull request #2198: Add support to configure
subscription name for sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2198
### Motivation
User can have requirement to create subscription on input-source topic with
a specific name only.
rdhabalia commented on a change in pull request #2192: Rename Netty native libs
to reflect shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#discussion_r203531605
##
File path: src/rename-netty-native-libs.sh
##
@@ -0,0 +1,54 @@
+#!/bin/bash
+#
+#
sijie commented on issue #2013: Unable to consume messages from a partition
URL:
https://github.com/apache/incubator-pulsar/issues/2013#issuecomment-406072137
@bardock it seems that bookie throttles the read requests. did you see any
increased read traffic from bookkeeper's metrics? it
rdhabalia opened a new pull request #2199: Cleanup unnecessary casting and
creating variables
URL: https://github.com/apache/incubator-pulsar/pull/2199
### Motivation
I think due to recent refactoring some of the minor code which is not
necessary to keep. so, cleaning up
sijie commented on issue #954: IPV6 support not available
URL:
https://github.com/apache/incubator-pulsar/issues/954#issuecomment-406087187
I think this is related to #1949
This is an automated message from the Apache Git
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406116727
retest this please
This is an automated
sijie commented on issue #983: optionally expose topic level metrics to
prometheus
URL:
https://github.com/apache/incubator-pulsar/issues/983#issuecomment-406084538
This is already implemented in #1064
This is an automated
merlimat commented on a change in pull request #2192: Rename Netty native libs
to reflect shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#discussion_r203544604
##
File path: src/rename-netty-native-libs.sh
##
@@ -0,0 +1,54 @@
+#!/bin/bash
+#
+#
sijie commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-406071521
retest this please
This is an automated message from the
sijie commented on issue #2195: Can't start Global Zookeeper in a Multi-cluster
setup
URL:
https://github.com/apache/incubator-pulsar/issues/2195#issuecomment-406081542
@dsambandam yes global-zookeeper is renamed to configuration-store since 2.0
release.
zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer
URL: https://github.com/apache/incubator-pulsar/pull/1996#issuecomment-406107756
@merlimat @ivankelly Thanks for the comments, It is updated, please help
review it.
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406116746
retest this please
This is an automated
rdhabalia opened a new pull request #2201: function-instance lookup: retrieve
function instance owner's workerId
URL: https://github.com/apache/incubator-pulsar/pull/2201
### Motivation
for administrative purpose, we need to know worker-owner of the
function-instance. so, adding
merlimat commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-406137576
retest this please
This is an automated message from the
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406137785
retest this please
This is an automated
rdhabalia commented on issue #2202: attach shaded artifact while deploying to
maven-repo
URL: https://github.com/apache/incubator-pulsar/pull/2202#issuecomment-406134453
@merlimat can we make change to get
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/asf-site by this push:
new c56cd3a Updated site at revision
sijie commented on issue #2185: Clean up sidebar menu under `Deployment`
URL: https://github.com/apache/incubator-pulsar/pull/2185#issuecomment-405968220
> Are we going to cleanup the different bare machine installation guides too?
Which ones? You mean ansible vs bare mentals?
ivankelly commented on issue #2185: Clean up sidebar menu under `Deployment`
URL: https://github.com/apache/incubator-pulsar/pull/2185#issuecomment-405977149
@sijie Yes.
Currently we have
- Deploy on Amazon Web Services (terraform + ansible)
- Single cluster on bare metal
ivankelly commented on a change in pull request #2179: Issue 2081: Improve
documentation about `--inputs` and `--customSerdeInputs` for pulsar functions
URL: https://github.com/apache/incubator-pulsar/pull/2179#discussion_r203431482
##
File path:
sijie commented on issue #2182: Issue 2110: get-retention returns
retentionSizeInMB as 0 when set to -1
URL: https://github.com/apache/incubator-pulsar/pull/2182#issuecomment-405839375
retest this please
This is an automated
rdhabalia commented on a change in pull request #2187: Removed the
`Context.ack(byte[] messageId)` and expose `Record` instead
URL: https://github.com/apache/incubator-pulsar/pull/2187#discussion_r203264915
##
File path: pulsar-io/kinesis/pom.xml
##
@@ -38,6 +38,13 @@
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 b3845b2 Moved Record interface as
merlimat closed pull request #2184: Moved Record interface as part of functions
api
URL: https://github.com/apache/incubator-pulsar/pull/2184
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 an automated email from the ASF dual-hosted git repository.
sijie 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 6ef9df9 augmenting protoschema with
merlimat opened a new pull request #2192: Rename Netty native libs to reflect
shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192
### Motivation
When using shaded artifacts, the Netty epoll and OpenSSL libraries are not
being loaded correctly and it falls back
sijie closed pull request #2181: augmenting protoschema with info for parsing
URL: https://github.com/apache/incubator-pulsar/pull/2181
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
dsambandam commented on issue #2195: Can't start Global Zookeeper in a
Multi-cluster setup
URL:
https://github.com/apache/incubator-pulsar/issues/2195#issuecomment-406017648
Also, global zookeeper option isn't supported in the pulsar daemon script
and wanted to confirm if running
merlimat commented on issue #2193: REST Api returns "405 Method not allowed"
URL:
https://github.com/apache/incubator-pulsar/issues/2193#issuecomment-406031223
@ratcashdev Created https://github.com/apache/incubator-pulsar/pull/2196 for
the doc fixes. Check the warning around the
dsambandam commented on issue #2037: Initialization of cluster meta data fails
while setting up a multi cluster pulsar instance
URL:
https://github.com/apache/incubator-pulsar/issues/2037#issuecomment-406018226
Apologies for the delays and I was travelling last week. We can close this
dsambandam commented on issue #2037: Initialization of cluster meta data fails
while setting up a multi cluster pulsar instance
URL:
https://github.com/apache/incubator-pulsar/issues/2037#issuecomment-406019311
Still having issues starting global zookeeper, so created a ticket to track
grantwwu commented on issue #2193: REST Api returns "405 Method not allowed"
URL:
https://github.com/apache/incubator-pulsar/issues/2193#issuecomment-406022885
#2174 relevant
This is an automated message from the Apache Git
merlimat opened a new pull request #2196: Improved dashboard documentation with
warning on advertised address
URL: https://github.com/apache/incubator-pulsar/pull/2196
### Motivation
The dashboard docs should warn user of setting the proper advertised address
on standalone service
merlimat commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-406031522
retest this please
This is an automated message from the
zhaijack commented on a change in pull request #1996: Cpp client: add
multiTopicsConsumer
URL: https://github.com/apache/incubator-pulsar/pull/1996#discussion_r203314440
##
File path: pulsar-client-cpp/lib/MultiTopicsConsumerImpl.cc
##
@@ -0,0 +1,621 @@
+/**
+ * Licensed
zhaijack commented on a change in pull request #1996: Cpp client: add
multiTopicsConsumer
URL: https://github.com/apache/incubator-pulsar/pull/1996#discussion_r203314440
##
File path: pulsar-client-cpp/lib/MultiTopicsConsumerImpl.cc
##
@@ -0,0 +1,621 @@
+/**
+ * Licensed
merlimat commented on issue #2177: Allow construction of c++ builtin auth
plugins via factory
URL: https://github.com/apache/incubator-pulsar/pull/2177#issuecomment-405873429
retest this please
This is an automated message
ivankelly commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-405863109
We need to start using osgi or something. All this shading is getting
ridiculous.
ivankelly removed a comment on issue #2192: Rename Netty native libs to reflect
shading prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-405863148
We need to start using osgi or something. All this shading is getting
ridiculous.
ivankelly commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-405863148
We need to start using osgi or something. All this shading is getting
ridiculous.
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 a5c339b Allow construction of c++
merlimat closed pull request #2177: Allow construction of c++ builtin auth
plugins via factory
URL: https://github.com/apache/incubator-pulsar/pull/2177
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
grantwwu commented on a change in pull request #2174: V2 doc changes
URL: https://github.com/apache/incubator-pulsar/pull/2174#discussion_r203455338
##
File path: site/docs/latest/admin-api/namespaces.md
##
@@ -144,7 +144,7 @@ test-tenant/ns2
REST API
-{%
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406012461
retest this please
This is an automated
grantwwu opened a new pull request #2194: WIP: Destination fix
URL: https://github.com/apache/incubator-pulsar/pull/2194
Some parts of the documentation reference `:destination`. The terminology
used elsewhere is `:topic`.
Depends on #2174.
There are a number of other
dsambandam opened a new issue #2195: Can't start Global Zookeeper in a
Multi-cluster setup
URL: https://github.com/apache/incubator-pulsar/issues/2195
Expected behavior
Global Zookeeper in a Multi-Cluster Setup doesn't start and fails when
started in foreground with Invalid
ivankelly commented on a change in pull request #2179: Issue 2081: Improve
documentation about `--inputs` and `--customSerdeInputs` for pulsar functions
URL: https://github.com/apache/incubator-pulsar/pull/2179#discussion_r203453439
##
File path:
merlimat commented on issue #2193: REST Api returns "405 Method not allowed"
URL:
https://github.com/apache/incubator-pulsar/issues/2193#issuecomment-406009908
Thanks for reporting @ratcashdev. I'll get a quick fix for the docs and the
dashboard code.
ivankelly commented on a change in pull request #2188: client configuration on
authentication page is out of date
URL: https://github.com/apache/incubator-pulsar/pull/2188#discussion_r203432701
##
File path: site/docs/latest/security/authorization.md
##
@@ -74,36 +74,19
ivankelly commented on a change in pull request #2188: client configuration on
authentication page is out of date
URL: https://github.com/apache/incubator-pulsar/pull/2188#discussion_r203433454
##
File path: site/docs/latest/security/authorization.md
##
@@ -74,36 +74,19
ivankelly commented on a change in pull request #2188: client configuration on
authentication page is out of date
URL: https://github.com/apache/incubator-pulsar/pull/2188#discussion_r203432802
##
File path: site/docs/latest/security/authorization.md
##
@@ -74,36 +74,19
ivankelly commented on a change in pull request #2188: client configuration on
authentication page is out of date
URL: https://github.com/apache/incubator-pulsar/pull/2188#discussion_r203435326
##
File path: site/docs/latest/security/tls.md
##
@@ -152,7 +152,7 @@ For more
sijie commented on a change in pull request #2179: Issue 2081: Improve
documentation about `--inputs` and `--customSerdeInputs` for pulsar functions
URL: https://github.com/apache/incubator-pulsar/pull/2179#discussion_r203435556
##
File path:
merlimat commented on a change in pull request #2174: V2 doc changes
URL: https://github.com/apache/incubator-pulsar/pull/2174#discussion_r203443654
##
File path: site/docs/latest/admin-api/namespaces.md
##
@@ -144,7 +144,7 @@ test-tenant/ns2
REST API
-{%
jerrypeng commented on issue #2197: unshade avro in pulsar-client-schema
URL: https://github.com/apache/incubator-pulsar/pull/2197#issuecomment-40615
retest this please
This is an automated message from the Apache Git
jerrypeng commented on issue #2200: support subscription-type to be passed in
sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2200#issuecomment-406160401
Can we not just infer from the processing guarantee. If the processing
guarantee is set At-least-once or
jerrypeng commented on issue #2197: unshade avro in pulsar-client-schema
URL: https://github.com/apache/incubator-pulsar/pull/2197#issuecomment-406162168
retest this please
This is an automated message from the Apache Git
merlimat commented on issue #2200: support subscription-type to be passed in
sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2200#issuecomment-406162067
> f the processing guarantee is set At-least-once or at-most-once, the
subscription type for the source should be
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406163614
retest this please
This is an automated
merlimat commented on issue #2192: Rename Netty native libs to reflect shading
prefix
URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-406163514
retest this please
This is an automated message from the
zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer
URL: https://github.com/apache/incubator-pulsar/pull/1996#issuecomment-406159737
retest this please
This is an automated message from the Apache Git
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406074860
retest this please
This is an automated
sijie commented on issue #976: Rate limiting for compaction reads
URL:
https://github.com/apache/incubator-pulsar/issues/976#issuecomment-406085364
Close this now. Will recreate the issue if we need more comprehensive rate
limiting on compaction.
sijie closed issue #976: Rate limiting for compaction reads
URL: https://github.com/apache/incubator-pulsar/issues/976
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406096097
retest this please
This is an automated
This is an automated email from the ASF dual-hosted git repository.
rdhabalia 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 e28ef1b Add support to configure
rdhabalia closed pull request #2198: Add support to configure subscription name
for sink-function
URL: https://github.com/apache/incubator-pulsar/pull/2198
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
rdhabalia removed a comment on issue #2199: Cleanup unnecessary casting and
creating variables
URL: https://github.com/apache/incubator-pulsar/pull/2199#issuecomment-406110490
retest this please
This is an automated message
rdhabalia commented on issue #2199: Cleanup unnecessary casting and creating
variables
URL: https://github.com/apache/incubator-pulsar/pull/2199#issuecomment-406127017
retest this please
This is an automated message from the
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects
URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406054206
@ivankelly There are same shading shading issues happening with tests here
could you take a look I can't seem to resolve it
bardock commented on issue #2013: Unable to consume messages from a partition
URL:
https://github.com/apache/incubator-pulsar/issues/2013#issuecomment-406054594
Sorry for the delay guys.
We faced the same issue past week.
This time we detected the issue 5 hours later, recreated
jerrypeng opened a new pull request #2197: unshade avro in pulsar-client-schema
URL: https://github.com/apache/incubator-pulsar/pull/2197
### Motivation
If users want to use Avro annotations with their POJOs, currently they would
have to use the our shaded version of avro because we
dsambandam commented on issue #2195: Can't start Global Zookeeper in a
Multi-cluster setup
URL:
https://github.com/apache/incubator-pulsar/issues/2195#issuecomment-406048364
confirmed running ./bin/pulsar-daemon start configuration-store starts
zookeeper on global ports .
Scripts
merlimat commented on issue #2191: Removed shading relocations for
Circe-checksum and lz4 libraries
URL: https://github.com/apache/incubator-pulsar/pull/2191#issuecomment-406037133
retest this please
This is an automated
merlimat commented on a change in pull request #2187: Removed the
`Context.ack(byte[] messageId)` and expose `Record` instead
URL: https://github.com/apache/incubator-pulsar/pull/2187#discussion_r203492473
##
File path: pulsar-io/kinesis/pom.xml
##
@@ -38,6 +38,13 @@
ratcashdev commented on issue #2193: REST Api returns "405 Method not allowed"
URL:
https://github.com/apache/incubator-pulsar/issues/2193#issuecomment-405915596
OK, got it. The docs on
https://pulsar.incubator.apache.org/docs/latest/admin-api/persistent-topics/
are not up-to-date.
jiazhai edited a comment on issue #1383: Unable to delete partitioned metadata
for a partitioned topic
URL:
https://github.com/apache/incubator-pulsar/issues/1383#issuecomment-405918077
@cckellogg. Seems this is function as designed, not able to reproduce it,
if using force delete.
jiazhai closed issue #1383: Unable to delete partitioned metadata for a
partitioned topic
URL: https://github.com/apache/incubator-pulsar/issues/1383
This is an automated message from the Apache Git Service.
To respond to
1 - 100 of 103 matches
Mail list logo
