fmiguelez commented on issue #12553:
URL: https://github.com/apache/pulsar/issues/12553#issuecomment-1097623177
@sijie can someone else take a look at 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
fmiguelez commented on issue #12553:
URL: https://github.com/apache/pulsar/issues/12553#issuecomment-1097622232
Also verified with **2.10.0**. Namespace reaches a situation when it can no
longer be removed.
```
$ curl -X DELETE
http://localhost:8081/admin/v2/namespaces/dbus/test?
wolfstudy closed pull request #759: [WIP][security] Upgrade jwt-go version to
3.2.1
URL: https://github.com/apache/pulsar-client-go/pull/759
--
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 speci
wolfstudy commented on code in PR #14527:
URL: https://github.com/apache/pulsar/pull/14527#discussion_r849124872
##
pulsar-function-go/go.mod:
##
@@ -3,15 +3,16 @@ module github.com/apache/pulsar/pulsar-function-go
go 1.13
require (
- github.com/apache/pulsar-client-go
eolivelli commented on code in PR #13833:
URL: https://github.com/apache/pulsar/pull/13833#discussion_r849123883
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/LedgerOffloaderStatsImpl.java:
##
@@ -0,0 +1,303 @@
+/**
+ * Licensed to the Apache Software Founda
wolfstudy commented on PR #759:
URL: https://github.com/apache/pulsar-client-go/pull/759#issuecomment-1097612153
@zymap @lhotari PTAL thanks.
--
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 s
wolfstudy commented on code in PR #14527:
URL: https://github.com/apache/pulsar/pull/14527#discussion_r849124872
##
pulsar-function-go/go.mod:
##
@@ -3,15 +3,16 @@ module github.com/apache/pulsar/pulsar-function-go
go 1.13
require (
- github.com/apache/pulsar-client-go
wolfstudy opened a new pull request, #759:
URL: https://github.com/apache/pulsar-client-go/pull/759
Signed-off-by: xiaolongran
Fixes #[apachepulsar: 14527](https://github.com/apache/pulsar/pull/14527)
### Motivation
There's an existing advisory about this library
eolivelli commented on PR #15136:
URL: https://github.com/apache/pulsar/pull/15136#issuecomment-1097615123
/pulsarbot rerun-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 to the spec
lhotari commented on code in PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#discussion_r849125281
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java:
##
@@ -1793,6 +1830,34 @@ protected void internalAsyncMarkDelete(final
Posit
lhotari commented on code in PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#discussion_r849118983
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java:
##
@@ -1793,6 +1830,34 @@ protected void internalAsyncMarkDelete(final
Posit
urfreespace opened a new pull request, #15148:
URL: https://github.com/apache/pulsar/pull/15148
Signed-off-by: Li Li
*(If this PR fixes a github issue, please add `Fixes #`.)*
[fix][website] building fail due to invalid tab error
*(or if this PR is one task of a
github-actions[bot] commented on PR #15147:
URL: https://github.com/apache/pulsar/pull/15147#issuecomment-1097602039
@gaozhangmin:Thanks for providing doc info!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
github-actions[bot] commented on PR #15147:
URL: https://github.com/apache/pulsar/pull/15147#issuecomment-1097599981
@gaozhangmin:Thanks for your contribution. For this PR, do we need to update
docs?
(The [PR template contains info about
doc](https://github.com/apache/pulsar/blob/master/
gaozhangmin opened a new pull request, #15147:
URL: https://github.com/apache/pulsar/pull/15147
### Motivation
There are so many unused parameters of method .
### Modifications
Remove unused parameters of method
### Documentation
Check the box below
john1337 opened a new issue, #15146:
URL: https://github.com/apache/pulsar/issues/15146
i am considering using delay message deliver using mq, found some
solutions: rabbitmq,rocketmq and pulsar, when i dig deep into pulsar i found
pulsar is using memory to realize delay deliver feature so
codelipenghui merged PR #14723:
URL: https://github.com/apache/pulsar/pull/14723
--
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: commits-unsubscr...@pul
eolivelli commented on PR #15142:
URL: https://github.com/apache/pulsar/pull/15142#issuecomment-1097576575
You can add the repository for bk 4.15.0 rc0
Like I did here
https://github.com/diennea/herddb/pull/784/files#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d
crossoverJie commented on code in PR #753:
URL: https://github.com/apache/pulsar-client-go/pull/753#discussion_r849090255
##
pulsar/consumer_test.go:
##
@@ -3017,3 +3017,59 @@ func TestEncryptDecryptRedeliveryOnFailure(t *testing.T)
{
assert.NotNil(t, msg)
consu
crossoverJie commented on code in PR #753:
URL: https://github.com/apache/pulsar-client-go/pull/753#discussion_r849090157
##
pulsar/consumer_test.go:
##
@@ -3017,3 +3017,59 @@ func TestEncryptDecryptRedeliveryOnFailure(t *testing.T)
{
assert.NotNil(t, msg)
consu
This is an automated email from the ASF dual-hosted git repository.
zhangmingao 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 19120e6b00e log transaction rest api error
gaozhangmin merged PR #15081:
URL: https://github.com/apache/pulsar/pull/15081
--
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: commits-unsubscr...@pulsa
mattisonchao commented on PR #15062:
URL: https://github.com/apache/pulsar/pull/15062#issuecomment-1097566849
Hi @HQebupt
This issue's root cause looks like we add `msgRateIn` and `msgThroughputIn`
status twice. the code is shown below:
Position 1:
https://github.com/apache/pulsar
Shoothzj closed pull request #14651: Bump netty version to 4.1.75
URL: https://github.com/apache/pulsar/pull/14651
--
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 unsubscrib
Shoothzj commented on PR #14651:
URL: https://github.com/apache/pulsar/pull/14651#issuecomment-1097563124
we need to update bookkeeper to bump netty version, closed this for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
Shawyeok commented on PR #15145:
URL: https://github.com/apache/pulsar/pull/15145#issuecomment-1097556976
/pulsarbot rerun-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 to the speci
congbobo184 commented on code in PR #15140:
URL: https://github.com/apache/pulsar/pull/15140#discussion_r849060142
##
pulsar-broker/src/main/java/org/apache/pulsar/broker/stats/prometheus/TopicStats.java:
##
@@ -42,6 +42,9 @@ class TopicStats {
long bytesOutCounter;
do
liudezhi2098 commented on PR #14756:
URL: https://github.com/apache/pulsar/pull/14756#issuecomment-1097542321
@gaozhangmin Please help review this 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
URL abov
wolfstudy commented on code in PR #734:
URL: https://github.com/apache/pulsar-client-go/pull/734#discussion_r849052252
##
pulsar/internal/topic_name.go:
##
@@ -107,6 +107,9 @@ func ParseTopicName(topic string) (*TopicName, error) {
}
func TopicNameWithoutPartitionPart(tn *To
wolfstudy commented on code in PR #734:
URL: https://github.com/apache/pulsar-client-go/pull/734#discussion_r849051366
##
pulsar/internal/topic_name.go:
##
@@ -107,6 +107,9 @@ func ParseTopicName(topic string) (*TopicName, error) {
}
func TopicNameWithoutPartitionPart(tn *To
Shawyeok opened a new pull request, #15145:
URL: https://github.com/apache/pulsar/pull/15145
### Motivation
Now it's not possible use `pulsar-client produce` send a message with `null`
body, but it's needed in some scenes, e.g. delete topic policy manually or test
topic compaction.
wolfstudy commented on PR #737:
URL: https://github.com/apache/pulsar-client-go/pull/737#issuecomment-1097527139
cc @zymap Please take a look thanks
--
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 t
wolfstudy commented on issue #752:
URL:
https://github.com/apache/pulsar-client-go/issues/752#issuecomment-1097524716
> Good point. I think we need to have a tool that automatically updates this
license file from the dependencies and we should check in the PR that it's
consistent.
L
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-client-go.git
from 0b0b312 Dlq producer on topic with schema (#723)
add 5ced071 fix annotation typo (#758)
No new revision
wolfstudy merged PR #758:
URL: https://github.com/apache/pulsar-client-go/pull/758
--
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: commits-unsubscr...@p
wolfstudy commented on PR #753:
URL: https://github.com/apache/pulsar-client-go/pull/753#issuecomment-1097524096
> @shileiyu It's a good idea. If `RetryEnable==true` in the Java SDK, a
default policy is created. https://user-images.githubusercontent.com/15684156/162584451-94974976-1a88-465f-
wolfstudy commented on code in PR #753:
URL: https://github.com/apache/pulsar-client-go/pull/753#discussion_r849044794
##
pulsar/consumer_test.go:
##
@@ -3017,3 +3017,59 @@ func TestEncryptDecryptRedeliveryOnFailure(t *testing.T)
{
assert.NotNil(t, msg)
consumer
wolfstudy commented on PR #750:
URL: https://github.com/apache/pulsar-client-go/pull/750#issuecomment-1097521388
@shileiyu Greate jobs! Can we add unit test to cover this case?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
This is an automated email from the ASF dual-hosted git repository.
rxl pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git
The following commit(s) were added to refs/heads/master by this push:
new 0b0b312 Dlq producer on topic with schem
wolfstudy merged PR #723:
URL: https://github.com/apache/pulsar-client-go/pull/723
--
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: commits-unsubscr...@p
tjiuming commented on code in PR #13833:
URL: https://github.com/apache/pulsar/pull/13833#discussion_r849033916
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/LedgerOffloaderStats.java:
##
@@ -0,0 +1,115 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF)
BewareMyPower commented on PR #15144:
URL: https://github.com/apache/pulsar/pull/15144#issuecomment-1097502964
Currently, zero queue consumer cannot receive batched messages. It is
intended, otherwise it should be a consumer whose queue size is 1.
The `ZeroQueueConsumerImpl#receiveInd
This is an automated email from the ASF dual-hosted git repository.
penghui 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 bb7670b20d8 Fix MemoryLimitTest for bra
codelipenghui merged PR #15138:
URL: https://github.com/apache/pulsar/pull/15138
--
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: commits-unsubscr...@pul
codelipenghui commented on PR #15064:
URL: https://github.com/apache/pulsar/pull/15064#issuecomment-1097501546
@vagetablechicken could you please help check the failed `OWASP Dependency
Check` test? I'm not sure if it was introduced by new dependencies, maybe you
can try to rebase to the ma
gaozhangmin commented on code in PR #15139:
URL: https://github.com/apache/pulsar/pull/15139#discussion_r849028068
##
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java:
##
@@ -1443,10 +1443,10 @@ public void openLedgerComplete(ManagedLedger ledger,
gaozhangmin commented on code in PR #14756:
URL: https://github.com/apache/pulsar/pull/14756#discussion_r849026358
##
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##
@@ -2288,6 +2288,12 @@ public class ServiceConfiguration implements
Pu
gaozhangmin commented on code in PR #14756:
URL: https://github.com/apache/pulsar/pull/14756#discussion_r849026358
##
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##
@@ -2288,6 +2288,12 @@ public class ServiceConfiguration implements
Pu
HQebupt commented on code in PR #15139:
URL: https://github.com/apache/pulsar/pull/15139#discussion_r849026303
##
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java:
##
@@ -1443,10 +1443,10 @@ public void openLedgerComplete(ManagedLedger ledger,
Obj
gaozhangmin commented on PR #15139:
URL: https://github.com/apache/pulsar/pull/15139#issuecomment-1097495881
LGTM, I'd like to point out that `createNonPersistentTopic` had fix this
bug already.
https://github.com/apache/pulsar/blob/a4c4aea993aabab5231d4136b7eba366bee9e778/pulsar-broker
nodece commented on PR #14798:
URL: https://github.com/apache/pulsar/pull/14798#issuecomment-1097494862
@michaeljmarshall Good catch, this PR has been changed, could you review
this PR again?
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
zymap commented on PR #15063:
URL: https://github.com/apache/pulsar/pull/15063#issuecomment-1097493054
@hangc0276 @horizonzy Could you please take another look?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
This is an automated email from the ASF dual-hosted git repository.
penghui 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 a4c4aea993a [fix][security] Remove log4j for CV
codelipenghui merged PR #15109:
URL: https://github.com/apache/pulsar/pull/15109
--
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: commits-unsubscr...@pul
jiazhai commented on PR #14756:
URL: https://github.com/apache/pulsar/pull/14756#issuecomment-1097482854
[cpp
test](https://github.com/apache/pulsar/runs/5983558878?check_suite_focus=true)
failed with error, restart the ci
```
[225/287] ProducerTest.testSendAsyncCloseAsyncConc
Anonymitaet commented on code in PR #14398:
URL: https://github.com/apache/pulsar/pull/14398#discussion_r848998528
##
site2/website/release-notes.md:
##
@@ -1,6 +1,301 @@
## Apache Pulsar Release Notes
+### 2.10.0
+ 2022-04-13
+
+### Important notice
+- Remove -XX:-Resi
github-actions[bot] commented on PR #15144:
URL: https://github.com/apache/pulsar/pull/15144#issuecomment-1097476240
@horizonzy:Thanks for providing doc info!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
github-actions[bot] commented on PR #15144:
URL: https://github.com/apache/pulsar/pull/15144#issuecomment-1097474886
@horizonzy:Thanks for your contribution. For this PR, do we need to update
docs?
(The [PR template contains info about
doc](https://github.com/apache/pulsar/blob/master/.g
horizonzy commented on PR #15144:
URL: https://github.com/apache/pulsar/pull/15144#issuecomment-1097471001
@BewareMyPower @michaeljmarshall cc
--
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
horizonzy opened a new pull request, #15144:
URL: https://github.com/apache/pulsar/pull/15144
### Motivation
Handle notifyPendingReceivedCallback in internalPinnedExecutor.
### Documentation
- [ ] `no-need-doc`
--
This is an automated message from the Apache Git Service.
This is an automated email from the ASF dual-hosted git repository.
guangning pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-manager.git
The following commit(s) were added to refs/heads/master by this push:
new 4ab8811 Add support for casdoor (#44
Anonymitaet commented on PR #15140:
URL: https://github.com/apache/pulsar/pull/15140#issuecomment-1097468432
@momo-jun a soft reminder: here is a PR w/ doc-required label, could u pls
follow up? Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message,
liangyepianzhou closed pull request #14911: [Fix][Transaction] Optimize log
when create transaction systemTopic
URL: https://github.com/apache/pulsar/pull/14911
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
horizonzy closed issue #15106: message maybe discard when pending receives all
done.
URL: https://github.com/apache/pulsar/issues/15106
--
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 c
horizonzy commented on PR #15107:
URL: https://github.com/apache/pulsar/pull/15107#issuecomment-1097465147
Thanks all reviewd, I got it. This enhancement looks unnecessary, close it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
horizonzy closed pull request #15107: [enhance] [client-consumer] Enhance
robustness in consumer executeNotifyCallback.
URL: https://github.com/apache/pulsar/pull/15107
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
codelipenghui commented on PR #14756:
URL: https://github.com/apache/pulsar/pull/14756#issuecomment-1097462498
@eolivelli Please help review this 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
URL above
liudezhi2098 commented on PR #15129:
URL: https://github.com/apache/pulsar/pull/15129#issuecomment-1097457422
> Thanks, @liudezhi2098 for the help. Why this change will fix this issue?
Would you please help provide more information in the PR description? such as
the reason behind the origin
liudezhi2098 commented on code in PR #15121:
URL: https://github.com/apache/pulsar/pull/15121#discussion_r848991282
##
conf/broker.conf:
##
@@ -767,6 +767,9 @@ saslJaasClientAllowedIds=
# Default value `SaslConstants.JAAS_DEFAULT_BROKER_SECTION_NAME`, which is
"Broker".
saslJ
codelipenghui commented on code in PR #15121:
URL: https://github.com/apache/pulsar/pull/15121#discussion_r848990644
##
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##
@@ -1438,6 +1438,12 @@ public class ServiceConfiguration implements
jiazhai commented on PR #15129:
URL: https://github.com/apache/pulsar/pull/15129#issuecomment-1097453543
Thanks, @liudezhi2098 for the help. Why this change will fix this issue?
Would you please help provide more information in the PR description? such as
the reason behind the original issu
This is an automated email from the ASF dual-hosted git repository.
penghui 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 97b7e77a907 fix debug log authenticate role err
codelipenghui merged PR #14784:
URL: https://github.com/apache/pulsar/pull/14784
--
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: commits-unsubscr...@pul
github-actions[bot] commented on PR #15143:
URL: https://github.com/apache/pulsar/pull/15143#issuecomment-1097414878
@rdhabalia:Thanks for your contribution. For this PR, do we need to update
docs?
(The [PR template contains info about
doc](https://github.com/apache/pulsar/blob/master/.g
rdhabalia opened a new pull request, #15143:
URL: https://github.com/apache/pulsar/pull/15143
### Motivation
Pulsar readers can also read large message and can utilize message chunking
feature. however, reader doesn't have chunking config to tune message-chunking
feature based on applica
dlg99 opened a new pull request, #15142:
URL: https://github.com/apache/pulsar/pull/15142
### Modifications
DRAFT. do not merge.
Upgraded BK to 4.15 (4.16-snapshot currently, as a placeholder for rc1 of
4.15)
Upgraded dependendencies, where applicable (license check to follo
github-actions[bot] commented on PR #15141:
URL: https://github.com/apache/pulsar/pull/15141#issuecomment-1097277313
@rdhabalia:Thanks for your contribution. For this PR, do we need to update
docs?
(The [PR template contains info about
doc](https://github.com/apache/pulsar/blob/master/.g
rdhabalia opened a new pull request, #15141:
URL: https://github.com/apache/pulsar/pull/15141
### Motivation
Right now, broker gives below error while deleting empty namespace without
any topic under the namespace but namespace was having partitioned-topic
metadata.
```
pulsar-admi
michaeljmarshall commented on code in PR #14798:
URL: https://github.com/apache/pulsar/pull/14798#discussion_r848710067
##
pulsar-common/src/test/java/org/apache/pulsar/common/util/TrustManagerProxyTest.java:
##
@@ -0,0 +1,48 @@
+/**
+ * Licensed to the Apache Software Foundatio
michaeljmarshall commented on PR #15107:
URL: https://github.com/apache/pulsar/pull/15107#issuecomment-1097037259
@horizonzy - given that there is no race because the use of a single thread
for updating `pendingRecieves` is intentionally part of this class's design, I
don't think there is a
This is an automated email from the ASF dual-hosted git repository.
urfree pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pulsar-site.git
The following commit(s) were added to refs/heads/main by this push:
new aacd25ffc9 Docs sync done from apache/pulsar(#e
github-actions[bot] commented on PR #15140:
URL: https://github.com/apache/pulsar/pull/15140#issuecomment-1097018691
@tjiuming:Thanks for providing doc info!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
github-actions[bot] commented on PR #15140:
URL: https://github.com/apache/pulsar/pull/15140#issuecomment-1097018239
@tjiuming:Thanks for your contribution. For this PR, do we need to update
docs?
(The [PR template contains info about
doc](https://github.com/apache/pulsar/blob/master/.gi
tjiuming commented on PR #15140:
URL: https://github.com/apache/pulsar/pull/15140#issuecomment-1096997618
Tests to be completed
--
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 commen
tjiuming opened a new pull request, #15140:
URL: https://github.com/apache/pulsar/pull/15140
### Motivation
Add metrics for transaction
--
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
dave2wave commented on issue #14934:
URL: https://github.com/apache/pulsar/issues/14934#issuecomment-1096994334
Do your credentials have a maximum messages per month?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
michaeljmarshall commented on PR #15122:
URL: https://github.com/apache/pulsar/pull/15122#issuecomment-1096985919
@cbornet - in that case, I don't think this PR should be labeled as fixing
#15118. It seems you've identified two problems, and we should fix both.
One question about your
michaeljmarshall commented on code in PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#discussion_r848638415
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java:
##
@@ -1793,6 +1830,34 @@ protected void internalAsyncMarkDelete(fin
michaeljmarshall commented on code in PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#discussion_r848638415
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java:
##
@@ -1793,6 +1830,34 @@ protected void internalAsyncMarkDelete(fin
lhotari commented on PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#issuecomment-1096955264
> > **This is wrong since the meaning of the field is not what one would
expect it to be.**
>
> @lhotari - are you able to describe the meaning of the field more?
I was maki
AnonHxy opened a new pull request, #15139:
URL: https://github.com/apache/pulsar/pull/15139
Fixs #15131
### Motivation
Fix #15131
### Modifications
`topics` remove in another thread
### Verifying this change
- [x] Make sure that the change passe
michaeljmarshall commented on PR #15067:
URL: https://github.com/apache/pulsar/pull/15067#issuecomment-1096925368
> **This is wrong since the meaning of the field is not what one would
expect it to be.**
@lhotari - are you able to describe the meaning of the field more?
--
This is
codelipenghui opened a new pull request, #15138:
URL: https://github.com/apache/pulsar/pull/15138
Fixed the failed MemoryLimitTest for branch-2.8
```
Error: Tests run: 8, Failures: 2, Errors: 0, Skipped: 6, Time elapsed:
16.627 s <<< FAILURE! - in org.apache.pulsar.client.api.Memo
liangyepianzhou commented on issue #15073:
URL: https://github.com/apache/pulsar/issues/15073#issuecomment-1096915534
I think we should let maxAckPosition as key, and logPosition as value, that
is because：
1. They are all incremental
2. maxAckPosition can be same, but logPosition alway
liangyepianzhou opened a new pull request, #15137:
URL: https://github.com/apache/pulsar/pull/15137
Master Issue: https://github.com/apache/pulsar/issues/15073
### Motivation
Reduce the memory occupied by metadataPositions and avoid OOM
### Modification
Regularly store a small amo
AnonHxy commented on issue #15131:
URL: https://github.com/apache/pulsar/issues/15131#issuecomment-1096897621
I will try to fix this issue
--
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 spe
michaeljmarshall commented on PR #15084:
URL: https://github.com/apache/pulsar/pull/15084#issuecomment-1096885089
Rebased to resolve conflicts. I didn't make any code changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
eolivelli opened a new pull request, #15136:
URL: https://github.com/apache/pulsar/pull/15136
### Motivation
In some test environment I have seen a problem using Apache HC based Driver.
I have filed this bug https://issues.apache.org/jira/browse/JCLOUDS-1605
Unfortunately the JCl
michaeljmarshall closed pull request #15044: [branch-2.8] Fix license files
URL: https://github.com/apache/pulsar/pull/15044
--
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
michaeljmarshall commented on PR #15044:
URL: https://github.com/apache/pulsar/pull/15044#issuecomment-1096856816
Thank you @codelipenghui. Closing this, since the underlying issue is now
resolved. @nicoloboschi - is it correct to assume that we're seeing this on
branch-2.9 and branch-2.10?
1 - 100 of 184 matches
Mail list logo
