codelipenghui commented on issue #11652:
URL: https://github.com/apache/pulsar/issues/11652#issuecomment-92766
@srouthu1 Have you tried a new Pulsar Client version? I noticed the client
version is 2.6.0 which released almost 2 years ago.
--
This is an automated message from the
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 1d1f633 [docs] [ISSUE 11380] Update code example and description
(#11653)
add 43ded59 Fix race condition in
sijie closed issue #11605:
URL: https://github.com/apache/pulsar/issues/11605
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
sijie merged pull request #11606:
URL: https://github.com/apache/pulsar/pull/11606
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from de86f4f [pulsar-admin] allow create functions with package URL
(#11666)
add 1d1f633 [docs] [ISSUE 11380]
sijie merged pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
sijie closed issue #11380:
URL: https://github.com/apache/pulsar/issues/11380
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from fbfbd0e Fix typos in concepts-transactions.md (#11676)
add de86f4f [pulsar-admin] allow create functions with
sijie closed issue #11665:
URL: https://github.com/apache/pulsar/issues/11665
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
sijie merged pull request #11666:
URL: https://github.com/apache/pulsar/pull/11666
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
murong00 commented on pull request #3823:
URL: https://github.com/apache/pulsar/pull/3823#issuecomment-899941008
Almost completed. I will resolve the conflicting files later and please help
to take a review if you are available, thanks.
--
This is an automated message from the Apache
equanz edited a comment on pull request #10534:
URL: https://github.com/apache/pulsar/pull/10534#issuecomment-899931286
@Vanlightly
> Moreover, it looks to me like the current strategy is already broken
seeing as the numbers won't always match (neither numbers nor order) such as
when
equanz commented on pull request #10534:
URL: https://github.com/apache/pulsar/pull/10534#issuecomment-899931286
@Vanlightly
> Moreover, it looks to me like the current strategy is already broken
seeing as the numbers won't always match (neither numbers nor order) such as
when
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689972100
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/CompactorMXBeanImpl.java
##
@@ -0,0 +1,125 @@
+/**
+ * Licensed to the Apache
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689972100
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/CompactorMXBeanImpl.java
##
@@ -0,0 +1,125 @@
+/**
+ * Licensed to the Apache
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689967367
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java
##
@@ -1903,9 +1905,28 @@ public
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689967293
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java
##
@@ -145,14 +149,17 @@ private void
xige-16 commented on issue #593:
URL:
https://github.com/apache/pulsar-client-go/issues/593#issuecomment-899925496
> Which client version are you using?
v0.5.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689964478
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java
##
@@ -145,14 +149,17 @@ private void
Technoboy- commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689963705
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/CompactorMXBeanImpl.java
##
@@ -0,0 +1,125 @@
+/**
+ * Licensed to the Apache
Anonymitaet edited a comment on pull request #11263:
URL: https://github.com/apache/pulsar/pull/11263#issuecomment-899907330
Doc is done here: https://github.com/apache/pulsar/pull/11602
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Anonymitaet commented on pull request #11263:
URL: https://github.com/apache/pulsar/pull/11263#issuecomment-899907330
Doc: https://github.com/apache/pulsar/pull/11602
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
Anonymitaet commented on pull request #11238:
URL: https://github.com/apache/pulsar/pull/11238#issuecomment-899906463
> > @Zhen-hao thanks for your great work. Would you like to add docs
accordingly? Then you can ping me to review, thanks
>
> That's the plan. I was overwhelmed by
This is an automated email from the ASF dual-hosted git repository.
liuyu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from b8dce10 Replace bookkeper with bookkeeper in bin/bookkeeper (#11675)
add fbfbd0e Fix typos in
Anonymitaet merged pull request #11676:
URL: https://github.com/apache/pulsar/pull/11676
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
Anonymitaet commented on pull request #11666:
URL: https://github.com/apache/pulsar/pull/11666#issuecomment-899903588
> @Anonymitaet thanks, we can start the doc once this PR is merged.
@freeznet please ping me if you submit a doc PR, thanks
--
This is an automated message from
cckellogg commented on issue #593:
URL:
https://github.com/apache/pulsar-client-go/issues/593#issuecomment-899895682
Which client version are you using?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
oversearch commented on pull request #11668:
URL: https://github.com/apache/pulsar/pull/11668#issuecomment-899865518
Still working on this... built it in clang and found some more warnings that
popped up which are actually potentially serious bugs (dangling references).
Trying to
bharanic-dev commented on pull request #11679:
URL: https://github.com/apache/pulsar/pull/11679#issuecomment-899863904
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
bharanic-dev commented on pull request #11679:
URL: https://github.com/apache/pulsar/pull/11679#issuecomment-899820437
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
michaeljmarshall commented on pull request #11468:
URL: https://github.com/apache/pulsar/pull/11468#issuecomment-899816400
@jerrypeng - thank you for taking a look. It seems to me that solution 2 is
more consistent with the way the broker and proxy load their configurations.
Let me know
eolivelli commented on issue #11548:
URL: https://github.com/apache/pulsar/issues/11548#issuecomment-899815526
So your problem is that you cannot disable non TLS ports?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
zbentley commented on issue #11324:
URL: https://github.com/apache/pulsar/issues/11324#issuecomment-899794441
This may be fixed by https://github.com/apache/pulsar/issues/11203; if
convenient, could you re-test?
--
This is an automated message from the Apache Git Service.
To respond to
hpvd commented on issue #11677:
URL: https://github.com/apache/pulsar/issues/11677#issuecomment-899735864
@eolivelli
many thanks for pointing to this!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
michaeljmarshall commented on pull request #11172:
URL: https://github.com/apache/pulsar/pull/11172#issuecomment-899734952
@hangc0276 - I believe this commit should be cherry picked to `branch-2.7`
as well.
--
This is an automated message from the Apache Git Service.
To respond to the
bharanic-dev commented on pull request #11582:
URL: https://github.com/apache/pulsar/pull/11582#issuecomment-899712869
LGTM
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
eolivelli commented on issue #11677:
URL: https://github.com/apache/pulsar/issues/11677#issuecomment-899712681
You may be interested in this work I am doing about Performance regression
testing for Pulsar
https://www.youtube.com/watch?v=NZnlLBT3BBY
bharanic-dev commented on pull request #11582:
URL: https://github.com/apache/pulsar/pull/11582#issuecomment-899712433
@leizhiyuan Thank you for taking care of this. Passing the Executor was one
of the tasks in my TODO list (which I why I left the comment in the code). Glad
you were able
bharanic-dev commented on pull request #11679:
URL: https://github.com/apache/pulsar/pull/11679#issuecomment-899696260
@jerrypeng PTAL when you get a chance.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
bharanic-dev opened a new pull request #11679:
URL: https://github.com/apache/pulsar/pull/11679
Fixes #11678
### Motivation
terminateFunction does not clean up batch source intermediate topics,
resulting in topic leak.
### Modifications
If an action does not
rdhabalia commented on a change in pull request #11671:
URL: https://github.com/apache/pulsar/pull/11671#discussion_r689722778
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BacklogQuotaManagerTest.java
##
@@ -1211,5 +1218,45 @@ public void
merlimat commented on a change in pull request #11668:
URL: https://github.com/apache/pulsar/pull/11668#discussion_r689719433
##
File path: pulsar-client-cpp/lib/Commands.cc
##
@@ -644,6 +644,7 @@ std::string Commands::messageType(BaseCommand_Type type) {
return
bharanic-dev opened a new issue #11678:
URL: https://github.com/apache/pulsar/issues/11678
**Describe the bug**
The cleanupBatchSource() tries to delete the intermediate topic here:
oversearch commented on pull request #11668:
URL: https://github.com/apache/pulsar/pull/11668#issuecomment-899656153
Note that this PR is not quite ready yet (hence the "draft"). I just wanted
to run it through the CI checks and not lose the message I typed up. I hadn't
compiled in
oversearch commented on a change in pull request #11668:
URL: https://github.com/apache/pulsar/pull/11668#discussion_r689693314
##
File path: pulsar-client-cpp/lib/Commands.cc
##
@@ -644,6 +644,7 @@ std::string Commands::messageType(BaseCommand_Type type) {
return
oversearch commented on a change in pull request #11668:
URL: https://github.com/apache/pulsar/pull/11668#discussion_r689686002
##
File path: pulsar-client-cpp/CMakeLists.txt
##
@@ -288,6 +288,10 @@ if (NOT APPLE AND NOT MSVC)
set(CMAKE_CXX_FLAGS_PYTHON
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch branch-2.8
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/branch-2.8 by this push:
new 118ad6c If a topic has compaction
gaoran10 commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689612419
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/CompactorMXBeanImpl.java
##
@@ -0,0 +1,125 @@
+/**
+ * Licensed to the Apache
gaoran10 commented on a change in pull request #11564:
URL: https://github.com/apache/pulsar/pull/11564#discussion_r689607897
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/compaction/CompactorMXBeanImpl.java
##
@@ -0,0 +1,125 @@
+/**
+ * Licensed to the Apache
hpvd opened a new issue #11677:
URL: https://github.com/apache/pulsar/issues/11677
**Is your enhancement request related to a problem? Please describe.**
When Pulsar grew, it always becomes a little more complex with more lines of
code.
Many of these steps could influence the
wuzhanpeng commented on pull request #11606:
URL: https://github.com/apache/pulsar/pull/11606#issuecomment-899438017
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
eolivelli commented on pull request #3823:
URL: https://github.com/apache/pulsar/pull/3823#issuecomment-899412521
@murong00 what is the status of this work ?
it looks like an interesting addition to Pulsar IO
--
This is an automated message from the Apache Git Service.
To respond to
eolivelli commented on pull request #7329:
URL: https://github.com/apache/pulsar/pull/7329#issuecomment-899412137
This patch is probably superseded by the recent changes in the ElasticSearch
connector.
--
This is an automated message from the Apache Git Service.
To respond to the
wuzhanpeng commented on pull request #11606:
URL: https://github.com/apache/pulsar/pull/11606#issuecomment-899404068
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
Vanlightly commented on pull request #10534:
URL: https://github.com/apache/pulsar/pull/10534#issuecomment-899386900
@equanz Let me know if I understand correctly. We publish aggregated stats,
where we aggregate by partitioned producer, and optionally we also publish
stats per partition
wuzhanpeng commented on a change in pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#discussion_r689395561
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiTest2.java
##
@@ -1472,6 +1474,60 @@ public void
eolivelli commented on a change in pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#discussion_r689390887
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiTest2.java
##
@@ -1472,6 +1474,60 @@ public void
wuzhanpeng commented on pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#issuecomment-899367078
@eolivelli @hangc0276 Sorry for the late. I supplemented a test case. PTAL~
--
This is an automated message from the Apache Git Service.
To respond to the message, please
wuzhanpeng commented on pull request #11606:
URL: https://github.com/apache/pulsar/pull/11606#issuecomment-899366052
> LGTM! could you please add some tests for it?
Sorry for the late. I supplemented a test case. PTAL~ @congbobo184
--
This is an automated message from the Apache
eolivelli commented on pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#issuecomment-899365068
IIUC @rdhabalia left some comments on the ML
@rdhabalia do you mind to official write your position about this PR ?
it also looks like that @jerrypeng initially
hangc0276 commented on pull request #11555:
URL: https://github.com/apache/pulsar/pull/11555#issuecomment-899358808
I doubt why not choose provide `OpStatsLogger`
(pulsar-broker/src/main/java/org/apache/pulsar/broker/stats/prometheus/metrics/DataSketchesOpStatsLogger.java)
to generate
hangc0276 commented on a change in pull request #11555:
URL: https://github.com/apache/pulsar/pull/11555#discussion_r689339248
##
File path:
pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/OffloadPoliciesImpl.java
##
@@ -172,6 +172,9 @@
@Configuration
wuzhanpeng commented on pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#issuecomment-899342199
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
This is an automated email from the ASF dual-hosted git repository.
rxl pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 9577b84 [Issue 11496][C++] Allow partitioned producers to start
lazily (#11570)
add b8dce10 Replace bookkeper
wolfstudy edited a comment on pull request #11675:
URL: https://github.com/apache/pulsar/pull/11675#issuecomment-899332618
> What about cherry pick to 2.8.1?
+1
@hangc0276 PTAL
--
This is an automated message from the Apache Git Service.
To respond to the message, please
wolfstudy merged pull request #11675:
URL: https://github.com/apache/pulsar/pull/11675
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
wolfstudy commented on pull request #11675:
URL: https://github.com/apache/pulsar/pull/11675#issuecomment-899332618
> What about cherry pick to 2.8.1?
+1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
sijia-w commented on issue #8671:
URL: https://github.com/apache/pulsar/issues/8671#issuecomment-899329010
@codelipenghui Any recommendations for the numbers?
From Ran: "We usually give recommendations according to the user
requirements in POC, then they run tests in this environment
wongxingjun commented on pull request #11540:
URL: https://github.com/apache/pulsar/pull/11540#issuecomment-899321683
> @wongxingjun all your PRs submit same doc changes for the same doc
version, please update versioned docs here
wongxingjun opened a new pull request #11676:
URL: https://github.com/apache/pulsar/pull/11676
fix typos
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe,
wongxingjun closed pull request #11673:
URL: https://github.com/apache/pulsar/pull/11673
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
wongxingjun closed pull request #11674:
URL: https://github.com/apache/pulsar/pull/11674
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
xige-16 opened a new issue #593:
URL: https://github.com/apache/pulsar-client-go/issues/593
Expected behavior
consume success after seekByTime
Actual behavior
After running the for loop a few times, step msg := <-consumer.Chan() will
be blocked.
Steps
sijia-w commented on pull request #11648:
URL: https://github.com/apache/pulsar/pull/11648#issuecomment-899301145
@codelipenghui Any suggestions for a new link for `Pulsar-java-tutorial`?
Many thanks.
--
This is an automated message from the Apache Git Service.
To respond to the
lhotari closed issue #11637:
URL: https://github.com/apache/pulsar/issues/11637
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
lhotari commented on issue #11637:
URL: https://github.com/apache/pulsar/issues/11637#issuecomment-899296289
Closing since this is fixed by #11656
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
sijia-w commented on pull request #11667:
URL: https://github.com/apache/pulsar/pull/11667#issuecomment-899294615
@Huanli-Meng Thanks for your comments. Updated. PTAL.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
Technoboy- commented on issue #11637:
URL: https://github.com/apache/pulsar/issues/11637#issuecomment-899287966
> This problem seems to be fixed by #11656 . @Technoboy- can you confirm?
Yes, I think so.
--
This is an automated message from the Apache Git Service.
To respond to the
wuzhanpeng commented on pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#issuecomment-899284986
/pulsarbot run-failure-checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
ca-simone-chiorazzo commented on issue #11203:
URL: https://github.com/apache/pulsar/issues/11203#issuecomment-899276799
The new 2.8.0.post0 release does not contain a wheel for many Linux,
breaking dockerized application that uses the python pulsar client. Are you
aware of this issue?
lhotari commented on pull request #10918:
URL: https://github.com/apache/pulsar/pull/10918#issuecomment-899269946
@eolivelli I rebased this PR. Let's see what the test results are.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
eolivelli commented on pull request #10918:
URL: https://github.com/apache/pulsar/pull/10918#issuecomment-899268506
@lhotari you want to try to rebase this patch to latest master ?
we committed the changes to DefaultImplementation about loading Pulsar
Client Impl (mostly) without
Anonymitaet commented on pull request #11540:
URL: https://github.com/apache/pulsar/pull/11540#issuecomment-899267940
@wongxingjun all your PRs submit same doc changes for the same doc version,
please update versioned docs here
wolfstudy opened a new pull request #11675:
URL: https://github.com/apache/pulsar/pull/11675
Signed-off-by: xiaolongran
### Motivation
![image](https://user-images.githubusercontent.com/20965307/129522786-1621dd8f-5d3a-48ab-82b3-437857a45ac8.png)
###
eolivelli commented on a change in pull request #11666:
URL: https://github.com/apache/pulsar/pull/11666#discussion_r689277408
##
File path:
pulsar-common/src/main/java/org/apache/pulsar/common/functions/Utils.java
##
@@ -34,7 +36,9 @@
public static boolean
85 matches
Mail list logo