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/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 05827ae remove pulsar-standalone image (#11657)
sijie merged pull request #11657:
URL: https://github.com/apache/pulsar/pull/11657
--
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:
Technoboy- commented on pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#issuecomment-898191394
/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.
hjf pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 0748ecb [C++] Fixing use-after-free and constructor bugs in
UnAckedMessageTrackerEnabled (#11630)
add e36233e
Jennifer88huang merged pull request #11658:
URL: https://github.com/apache/pulsar/pull/11658
--
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:
Technoboy- commented on pull request #11655:
URL: https://github.com/apache/pulsar/pull/11655#issuecomment-898161804
/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
Jennifer88huang opened a new pull request #11658:
URL: https://github.com/apache/pulsar/pull/11658
### Modifications
Remove the broken links and link to the third part website.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
freeznet commented on pull request #11623:
URL: https://github.com/apache/pulsar/pull/11623#issuecomment-898138663
@eolivelli I just created a PR https://github.com/apache/pulsar/pull/11657
to remove the pulsar-standalone image, PTAL when you have time, thanks.
--
This is an automated
freeznet opened a new pull request #11657:
URL: https://github.com/apache/pulsar/pull/11657
### Motivation
Currently, the pulsar-dashboard has been removed in
https://github.com/apache/pulsar/pull/11284, as the pulsar-standalone image is
required pulsar-dashboard and provide
Technoboy- opened a new pull request #11656:
URL: https://github.com/apache/pulsar/pull/11656
### Motivation
getNamespaceFromPoliciesPath(n.getPath()) may return empty, and result
NamespaceName.get throw IllegalArgumentException
### Documentation
This is code-optimized, no
gaoran10 commented on issue #11576:
URL: https://github.com/apache/pulsar/issues/11576#issuecomment-898109632
Maybe you could refer to Java client logic, the producer will register
schema before producing messages to make sure the schema saved in BookKeeper,
the register schema command
Anonymitaet commented on pull request #11648:
URL: https://github.com/apache/pulsar/pull/11648#issuecomment-898101919
@sijia-w @Jennifer88huang how about
https://opensourcelibs.com/lib/pulsar-java-tutorial?
--
This is an automated message from the Apache Git Service.
To respond to the
Technoboy- opened a new pull request #11655:
URL: https://github.com/apache/pulsar/pull/11655
### Motivation
- Check variable first instead of catch NPE.
### Documentation
This is code-optimized, no need to update the doc.
--
This is an automated message from the
Sunny-Island commented on issue #11561:
URL: https://github.com/apache/pulsar/issues/11561#issuecomment-898087304
This issue can be closed.
https://github.com/apache/pulsar/pull/11600
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Technoboy- commented on pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#issuecomment-898018381
/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
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##
@@ -243,7 +245,7 @@ protected
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##
@@ -243,7 +245,7 @@ protected
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##
@@ -243,7 +245,7 @@ protected
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688081880
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##
@@ -883,16 +887,21 @@ private void
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688077951
##
File path:
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java
##
@@ -83,6 +86,49 @@ public
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688076935
##
File path:
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java
##
@@ -83,6 +86,49 @@ public
jerrypeng commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r688064193
##
File path:
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java
##
@@ -83,6 +86,49 @@ public
acjohnson opened a new issue #11654:
URL: https://github.com/apache/pulsar/issues/11654
**Describe the bug**
After upgrading to Official Apache Pulsar 2.8.0 docker image we are now
seeing a flood of messages like this:
```
{"line":"22:48:42.086 [pulsar-client-io-51-1] ERROR
sijia-w commented on issue #11380:
URL: https://github.com/apache/pulsar/issues/11380#issuecomment-897947070
@Jennifer88huang I checked this page
https://pulsar.apache.org/docs/en/cookbooks-retention-expiry/#clear-backlog and
the `N/A`s seem to be empty returned messages. If they are not,
sijia-w commented on pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653#issuecomment-897946101
@Jennifer88huang Please help review the changes. Thank you.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from bf62bd9 [PIP-82] [pulsar-broker] Use the qualified NS-name in
RG-register/unregisters (#11643)
add 0748ecb
sijia-w opened a new pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653
Fixes #11380
It seems that the changed two file share common content on the usage of
message retention policies. Do we need to keep the examples of the commands in
both files?
--
This is
merlimat merged pull request #11630:
URL: https://github.com/apache/pulsar/pull/11630
--
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.
jerrypeng pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 683c2d6a Java Client: remove usage of reflection while using Pulsar
Implementation classes (#11636)
add
jerrypeng merged pull request #11643:
URL: https://github.com/apache/pulsar/pull/11643
--
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:
oversearch commented on pull request #11630:
URL: https://github.com/apache/pulsar/pull/11630#issuecomment-897906422
I wasn't able to reproduce the stack dump in the tests in my local
environment. Everything passes except for an obscure error about a
bytes/string type mismatch in the
ivankelly commented on pull request #11570:
URL: https://github.com/apache/pulsar/pull/11570#issuecomment-897892377
> For authn/authz backward compatibility reason, I think the internal
producer should be connected to broker before completing the creation of
partitioned producer(
kaushik-develop commented on pull request #11643:
URL: https://github.com/apache/pulsar/pull/11643#issuecomment-897866903
/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
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from eabf52f [Blog] Update Pulsar summit link (#11651)
add 683c2d6a Java Client: remove usage of reflection while
merlimat merged pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636
--
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:
merlimat commented on a change in pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636#discussion_r687966915
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java
##
@@ -0,0 +1,348 @@
+/**
+ *
eaba closed issue #11576:
URL: https://github.com/apache/pulsar/issues/11576
--
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:
eaba commented on issue #11576:
URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897831708
@gaoran10 thanks. Null schemaversion was the problem. I have it fixed now.
Thanks a lot!!
--
This is an automated message from the Apache Git Service.
To respond to the
oversearch commented on pull request #11630:
URL: https://github.com/apache/pulsar/pull/11630#issuecomment-897821212
Ah yes, sorry about leaving out the docs section. I had assumed leaving it
out would imply "no" but I'll make that explicit.
I'm also looking into the python test
ivankelly commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r687941161
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java
##
@@ -327,6 +333,14 @@ public boolean
oversearch commented on issue #11632:
URL: https://github.com/apache/pulsar/issues/11632#issuecomment-897818966
Thanks for the feedback guys. I'll proceed with the changes as time allows,
starting with the improved warnings.
With respect to the boost compatibility stuff: you're
kaushik-develop commented on a change in pull request #11643:
URL: https://github.com/apache/pulsar/pull/11643#discussion_r687935502
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceQuotaCalculatorImpl.java
##
@@ -60,7 +68,11 @@ public
kaushik-develop commented on a change in pull request #11643:
URL: https://github.com/apache/pulsar/pull/11643#discussion_r687935424
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java
##
@@ -233,66 +235,80 @@ public void
eolivelli commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897759754
I have run my tests and with this helm chart ZooKeeper works well with 3 pods
https://github.com/datastax/pulsar-helm-chart
--
This is an automated message from the Apache
LeBW commented on a change in pull request #11553:
URL: https://github.com/apache/pulsar/pull/11553#discussion_r687870069
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarCommandSenderImpl.java
##
@@ -243,8 +243,12 @@ public ChannelPromise
srouthu1 opened a new issue #11652:
URL: https://github.com/apache/pulsar/issues/11652
Expected behavior
I have created consumer with the following code.
Consumer consumer = client.newConsumer()
.topic("persistent://mytenant/myns/mytopic")
eolivelli commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897695832
@andrekramer1 this line seems relevant for upgrading ZK and Pulsar 2.8
eolivelli commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897690248
I am testing it now with my helm chartstay tuned
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
eolivelli commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897683866
which helm chart are you using ?
the "official" Apache Pulsar Helm chart was never updated to Pulsar 2.8.0
see this PR that has not been merged
hangc0276 commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897676723
Thanks for @eolivelli 's help, from the exception stack, i looks like
something wrong in zookeeper or the helm chart. Would you please paste detail
logs of zookeeper?
This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 36d5738 Add Docs for the new ElasticSearch sink features (#11602)
add eabf52f [Blog] Update Pulsar summit
eolivelli merged pull request #11651:
URL: https://github.com/apache/pulsar/pull/11651
--
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:
eolivelli commented on issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897664698
the NPE is here
Technoboy- commented on a change in pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#discussion_r687717002
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/web/ExceptionHandlerTest.java
##
@@ -0,0 +1,55 @@
+/**
+ * Licensed to the Apache
Technoboy- commented on a change in pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#discussion_r687715617
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java
##
@@ -159,11 +159,12 @@ public void addServlet(String path,
Technoboy- commented on a change in pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#discussion_r687715269
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java
##
@@ -159,11 +159,12 @@ public void addServlet(String path,
eolivelli commented on a change in pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650#discussion_r687710188
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java
##
@@ -159,11 +159,12 @@ public void addServlet(String path,
Technoboy- opened a new pull request #11650:
URL: https://github.com/apache/pulsar/pull/11650
Fixes #11649
### Documentation
This is a bug fix, and do not need to update doc.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Technoboy- opened a new issue #11649:
URL: https://github.com/apache/pulsar/issues/11649
**Describe the bug**
Cluster didn't respond error messages when hitting the resource limit
**To Reproduce**
Set the limit of number of tenants to 4, create a new tenant when there're
HistoryGift commented on issue #5099:
URL: https://github.com/apache/pulsar/issues/5099#issuecomment-897591799
I have meet the same problem with python script example, but it's ok with
java script
someone say it's because of downloadDIrectory in functions_worker.yaml has
no execute
315157973 commented on a change in pull request #11553:
URL: https://github.com/apache/pulsar/pull/11553#discussion_r687646847
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarCommandSenderImpl.java
##
@@ -243,8 +243,12 @@ public ChannelPromise
sijia-w commented on issue #11291:
URL: https://github.com/apache/pulsar/issues/11291#issuecomment-897583889
@Jennifer88huang @Anonymitaet Thanks for your feedback. Please help review
the pr linked.
--
This is an automated message from the Apache Git Service.
To respond to the message,
sijia-w commented on pull request #11648:
URL: https://github.com/apache/pulsar/pull/11648#issuecomment-897581541
@Anonymitaet @Jennifer88huang Please review the changes and I look forward
to your recommendation for the links. Thanks.
--
This is an automated message from the Apache Git
sijia-w opened a new pull request #11648:
URL: https://github.com/apache/pulsar/pull/11648
Fixes #11291
The links for `[Pulsar Java
tutorial](https://github.com/streamlio/pulsar-java-tutorial)` and relevant
content are broken. Is there any recommendations for what we should use
eolivelli commented on pull request #10485:
URL: https://github.com/apache/pulsar/pull/10485#issuecomment-897568305
@merlimat precommit check passed
--
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 issue #11070:
URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897567591
@hangc0276 I believe that we should fix this problem before cutting 2.8.1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
codelipenghui commented on pull request #11553:
URL: https://github.com/apache/pulsar/pull/11553#issuecomment-897539348
@eolivelli Please help review the PR again.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
abhilashmandaliya opened a new issue #11647:
URL: https://github.com/apache/pulsar/issues/11647
**Describe the bug**
A backlog quota in the broker is configured differently from the broker's
configuration file than the admin CLI. A configuration file accepts the limit
in GB only as
wuzhanpeng commented on pull request #11640:
URL: https://github.com/apache/pulsar/pull/11640#issuecomment-897528300
> Good work
> Can you add a test case?
Sorry for the late reply! I will add a test later.
--
This is an automated message from the Apache Git Service.
To respond
BewareMyPower commented on issue #11635:
URL: https://github.com/apache/pulsar/issues/11635#issuecomment-897527855
I also found this problem recently. It's flaky but the cause is hard to know
currently. Maybe a temporary solution is adding the retry logic for `python
pulsar_test.py`? I'm
This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 36d5738 Add Docs for the new ElasticSearch
eolivelli commented on pull request #11602:
URL: https://github.com/apache/pulsar/pull/11602#issuecomment-897496847
thank you @Anonymitaet !
--
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
eolivelli merged pull request #11602:
URL: 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 the
URL above to go to the specific comment.
To unsubscribe, e-mail:
naveenkusakula commented on issue #11646:
URL: https://github.com/apache/pulsar/issues/11646#issuecomment-897492852
I loaded deadLetterPolicy() using normal builder and than loadConfi()
without deadLetterPolicy still not able to load configuration like I mentioned
in option 2.
--
This
eolivelli commented on issue #11646:
URL: https://github.com/apache/pulsar/issues/11646#issuecomment-897490867
Unfortunately there are some other configuration options that are not
configurable via loadConf, for instance Authentication in the
PulsarClientBuilder.
I would like to
eolivelli commented on pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636#issuecomment-897475256
@merlimat I have removed the usage of Fully qualified class names.
I have answered to your question about classloading (I had to verify this in
the specs).
After
eolivelli commented on a change in pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636#discussion_r687529763
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java
##
@@ -0,0 +1,348 @@
+/**
+ *
BewareMyPower closed issue #11549:
URL: https://github.com/apache/pulsar/issues/11549
--
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:
BewareMyPower commented on issue #11549:
URL: https://github.com/apache/pulsar/issues/11549#issuecomment-897472062
It's fixed by #11575
--
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
BewareMyPower commented on issue #11632:
URL: https://github.com/apache/pulsar/issues/11632#issuecomment-897470434
LGTM, here're my comments.
> - Enable many more warnings (-Wall, -Wvla, -Wformat-security, etc)
> - Enforce warnings as errors to prevent changes being submitted with
equanz commented on pull request #11570:
URL: https://github.com/apache/pulsar/pull/11570#issuecomment-897463605
Thank you for your comments about PIP-79.
I have a few comments. I think these comments don't block merging.
For authn/authz backward compatibility reason, I think the
eolivelli commented on pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636#issuecomment-897456337
> @eolivelli thanks for providing doc-related info. If your PR does not need
to update doc, could you please help label the PR with `no-need-doc` in the
future? Thanks
eolivelli commented on a change in pull request #11636:
URL: https://github.com/apache/pulsar/pull/11636#discussion_r687509897
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java
##
@@ -0,0 +1,348 @@
+/**
+ *
naveenkusakula opened a new issue #11646:
URL: https://github.com/apache/pulsar/issues/11646
Not able to load "deadLetterPolicy" when using loadConf(Map
config) and not able to loadConf(Map config), after
deadLetterPolicy() is called in ConsumerBuilder.
**Steps to Reporoduce:**
This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 9088294 Sync up C error code with C++ (#11633)
add 402a88d Fix flaky tests for ElasticSearch Sink (#11638)
eolivelli merged pull request #11638:
URL: https://github.com/apache/pulsar/pull/11638
--
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:
eolivelli closed issue #11628:
URL: https://github.com/apache/pulsar/issues/11628
--
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:
ivankelly commented on pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#issuecomment-897428763
> Thanks for your contribution. For this PR, do we need to update docs?
>
> (The [PR template contains info about
ivankelly commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r687477622
##
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##
@@ -883,16 +887,21 @@ private void
ivankelly commented on a change in pull request #11627:
URL: https://github.com/apache/pulsar/pull/11627#discussion_r687476062
##
File path:
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java
##
@@ -83,6 +86,49 @@ public
gaoran10 edited a comment on issue #11576:
URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897407506
I think the root cause is the message metadata is the lake of
`schemaVersion` property. You could change the message serialization method to
add schema version in message
This is an automated email from the ASF dual-hosted git repository.
chenhang 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 dd796d8 fix flaky test in
This is an automated email from the ASF dual-hosted git repository.
chenhang 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 f918370 [Branch-2.8]fix license
hangc0276 merged pull request #11645:
URL: https://github.com/apache/pulsar/pull/11645
--
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:
hangc0276 opened a new pull request #11645:
URL: https://github.com/apache/pulsar/pull/11645
Related to #11644
--
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
LeBW commented on a change in pull request #11553:
URL: https://github.com/apache/pulsar/pull/11553#discussion_r687458847
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerEntryMetadataE2ETest.java
##
@@ -273,6 +274,78 @@ public void
gaoran10 commented on issue #11576:
URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897407506
I think the root cause is the message metadata is lake of schemaVersion
property. You could change the message serialization method to add schema
version in message metadata.
--
frankxieke commented on a change in pull request #11555:
URL: https://github.com/apache/pulsar/pull/11555#discussion_r687427116
##
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/LedgerOffloaderMXBeanImpl.java
##
@@ -0,0 +1,269 @@
+/**
+ * Licensed
frankxieke commented on a change in pull request #11555:
URL: https://github.com/apache/pulsar/pull/11555#discussion_r687426921
##
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/LedgerOffloaderMXBeanImpl.java
##
@@ -0,0 +1,269 @@
+/**
+ * Licensed
frankxieke commented on a change in pull request #11555:
URL: https://github.com/apache/pulsar/pull/11555#discussion_r687425798
##
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/LedgerOffloaderMXBean.java
##
@@ -0,0 +1,139 @@
+/**
+ * Licensed to the
1 - 100 of 103 matches
Mail list logo