xiaogang-sn commented on code in PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#discussion_r1161590735
##
docs/client-libraries-cluster-level-failover.md:
##
@@ -26,7 +26,8 @@ For more information about cluster-level failover, including
concepts, benefits,
*
RobertIndie commented on code in PR #309:
URL:
https://github.com/apache/pulsar-client-node/pull/309#discussion_r1161600987
##
src/Consumer.cc:
##
@@ -55,21 +56,34 @@ void Consumer::Init(Napi::Env env, Napi::Object exports) {
struct MessageListenerProxyData {
codelipenghui commented on code in PR #20040:
URL: https://github.com/apache/pulsar/pull/20040#discussion_r1161606167
##
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Consumer.java:
##
@@ -536,19 +537,38 @@ CompletableFuture
reconsumeLaterCumulativeAsync(Message
BewareMyPower commented on code in PR #20040:
URL: https://github.com/apache/pulsar/pull/20040#discussion_r1161609338
##
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Consumer.java:
##
@@ -536,19 +537,38 @@ CompletableFuture
reconsumeLaterCumulativeAsync(Message
BewareMyPower commented on code in PR #20009:
URL: https://github.com/apache/pulsar/pull/20009#discussion_r1161626720
##
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientBuilderImpl.java:
##
@@ -410,4 +410,27 @@ public ClientBuilder socks5ProxyPassword(String
BewareMyPower commented on code in PR #309:
URL:
https://github.com/apache/pulsar-client-node/pull/309#discussion_r1161643305
##
src/Consumer.cc:
##
@@ -55,21 +56,34 @@ void Consumer::Init(Napi::Env env, Napi::Object exports) {
struct MessageListenerProxyData {
jiangpengcheng commented on PR #2:
URL: https://github.com/apache/pulsar/pull/2#issuecomment-1501696923
/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
tisonkun commented on issue #20041:
URL: https://github.com/apache/pulsar/issues/20041#issuecomment-1501602244
Anyway, this seems to be a usage problem instead of an issue. Moved to the
Q discussion forum.
--
This is an automated message from the Apache Git Service.
To respond to the
GitHub user tisonkun added a comment to the discussion: [Bug] Failed to pull-up
docker-compose for broker, bookie, zk and pulsar-sql
Also, the root cause is:
```
pulsar-sql | at
org.apache.pulsar.client.admin.internal.PulsarAdminBuilderImpl.authentication(PulsarAdminBuilderImpl.java:94)
tisonkun closed issue #20041: [Bug] Failed to pull-up docker-compose for
broker, bookie, zk and pulsar-sql
URL: https://github.com/apache/pulsar/issues/20041
--
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
GitHub user svabra added a comment to the discussion: [Bug] Failed to pull-up
docker-compose for broker, bookie, zk and pulsar-sql
I was wondering. If the CLI is triggering the Gateway, which includes the REST
service according to this [presentation](https://youtu.be/DnXu0Q0mvTM?t=962),
then
GitHub user tisonkun added a comment to the discussion: [Bug] Failed to pull-up
docker-compose for broker, bookie, zk and pulsar-sql
The error logs said "IllegalArgumentException", maybe you can change:
```
pulsar.nar-extraction-directory=System.getProperty("java.io.tmpdir")
```
to
```
GitHub user tisonkun added a comment to the discussion: [Bug] Failed to pull-up
docker-compose for broker, bookie, zk and pulsar-sql
Anyway, this seems to be a usage problem instead of an issue. Moved to the Q
discussion forum.
GitHub link:
tisonkun commented on issue #20041:
URL: https://github.com/apache/pulsar/issues/20041#issuecomment-1501601837
Also, the root cause is:
```
pulsar-sql | at
org.apache.pulsar.client.admin.internal.PulsarAdminBuilderImpl.authentication(PulsarAdminBuilderImpl.java:94)
pulsar-sql |
Anonymitaet opened a new pull request, #510:
URL: https://github.com/apache/pulsar-site/pull/510
Fix https://github.com/apache/pulsar/issues/20051
- [x] `doc`
@hangc0276 thanks for your technical explanations!
Ping @D-2-Ed for review, thanks!
--
This is an automated
codelipenghui commented on code in PR #20009:
URL: https://github.com/apache/pulsar/pull/20009#discussion_r1161581756
##
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientBuilderImpl.java:
##
@@ -410,4 +410,27 @@ public ClientBuilder socks5ProxyPassword(String
Anonymitaet commented on PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#issuecomment-1501611850
Previews look good:
https://user-images.githubusercontent.com/50226895/230875130-02e09e54-89f0-4deb-aa70-679a6417f62d.png;>
Anonymitaet commented on issue #20051:
URL: https://github.com/apache/pulsar/issues/20051#issuecomment-1501613311
Fix in https://github.com/apache/pulsar-site/pull/510
--
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 PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#issuecomment-1501613011
For reviewers: I'll apply the same doc changes to 2.11.x and 2.10.x if this
PR LGTY.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Anonymitaet commented on PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#issuecomment-1501613951
@hangc0276 @xiaogang-sn Could you please review this PR from a technical
perspective? TIA!
--
This is an automated message from the Apache Git Service.
To respond to the message,
jdfrozen commented on issue #14982:
URL: https://github.com/apache/pulsar/issues/14982#issuecomment-1501651275
2.8.4 duplicated delayed messages when unload topic
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
Anonymitaet commented on code in PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#discussion_r1161615807
##
docs/client-libraries-cluster-level-failover.md:
##
@@ -26,7 +26,8 @@ For more information about cluster-level failover, including
concepts, benefits,
*
Anonymitaet commented on code in PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#discussion_r1161615807
##
docs/client-libraries-cluster-level-failover.md:
##
@@ -26,7 +26,8 @@ For more information about cluster-level failover, including
concepts, benefits,
*
BewareMyPower commented on code in PR #20040:
URL: https://github.com/apache/pulsar/pull/20040#discussion_r1161624425
##
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Consumer.java:
##
@@ -536,19 +537,38 @@ CompletableFuture
reconsumeLaterCumulativeAsync(Message
coderzc commented on code in PR #20031:
URL: https://github.com/apache/pulsar/pull/20031#discussion_r1161472533
##
conf/pulsar_tools_env.sh:
##
@@ -42,6 +42,18 @@
# PULSAR_GLOBAL_ZK_CONF=
# Extra options to be passed to the jvm
+# Discard parameter "-Xms" of $PULSAR_MEM,
Anonymitaet commented on issue #20036:
URL: https://github.com/apache/pulsar/issues/20036#issuecomment-1501469268
@momo-jun OK, thanks!
Updating the color text to black text is a better way since black text and
the image in NEXT and 2.11.x matches our current design style (while the
coderzc commented on code in PR #20031:
URL: https://github.com/apache/pulsar/pull/20031#discussion_r1161472533
##
conf/pulsar_tools_env.sh:
##
@@ -42,6 +42,18 @@
# PULSAR_GLOBAL_ZK_CONF=
# Extra options to be passed to the jvm
+# Discard parameter "-Xms" of $PULSAR_MEM,
tisonkun closed issue #19363: [Bug] pulsar-sink nested avro
URL: https://github.com/apache/pulsar/issues/19363
--
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,
tisonkun commented on issue #19363:
URL: https://github.com/apache/pulsar/issues/19363#issuecomment-1501489462
The repo link is broken and 2.9.x is end of life.
Closing as stale. But if it's still relevant to maintained version (ATM 2.10
and 2.11), please open a new issue with error
gaozhangmin opened a new pull request, #20052:
URL: https://github.com/apache/pulsar/pull/20052
### Motivation
Currently, Pulsar's Dynamic Configuration Feature only supports
cluster-level configuration, meaning that any changes made to the dynamic
configuration apply to all brokers
github-actions[bot] commented on PR #20052:
URL: https://github.com/apache/pulsar/pull/20052#issuecomment-1501492591
@gaozhangmin Please add the following content to your PR description and
select a checkbox:
```
- [ ] `doc`
- [ ] `doc-required`
- [ ] `doc-not-needed`
- [
poorbarcode commented on PR #18981:
URL: https://github.com/apache/pulsar/pull/18981#issuecomment-1501514465
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19618:
URL: https://github.com/apache/pulsar/pull/19618#issuecomment-1501513977
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19026:
URL: https://github.com/apache/pulsar/pull/19026#issuecomment-1501516224
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19626:
URL: https://github.com/apache/pulsar/pull/19626#issuecomment-1501515813
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #12351:
URL: https://github.com/apache/pulsar/pull/12351#issuecomment-1501517511
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #12599:
URL: https://github.com/apache/pulsar/pull/12599#issuecomment-1501518509
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19501:
URL: https://github.com/apache/pulsar/pull/19501#issuecomment-1501537704
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19348:
URL: https://github.com/apache/pulsar/pull/19348#issuecomment-1501542840
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
github-actions[bot] commented on PR #14286:
URL: https://github.com/apache/pulsar/pull/14286#issuecomment-1501569855
@congbobo184 Please add the following content to your PR description and
select a checkbox:
```
- [ ] `doc`
- [ ] `doc-required`
- [ ] `doc-not-needed`
- [
poorbarcode commented on PR #15144:
URL: https://github.com/apache/pulsar/pull/15144#issuecomment-1501570391
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
tisonkun commented on issue #19912:
URL: https://github.com/apache/pulsar/issues/19912#issuecomment-1501458624
Hi @asafm! Thanks for starting this thread.
I reviewed this proposal in two aspects.
## Content
For three journeys in the proposal, we have contents for two of
tisonkun commented on PR #20016:
URL: https://github.com/apache/pulsar/pull/20016#issuecomment-1501480354
Merging...
Thanks for your review.
--
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
tisonkun merged PR #20016:
URL: https://github.com/apache/pulsar/pull/20016
--
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.
tison 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 470b674016c [feat][sql] Bump Trino version to
poorbarcode commented on PR #20049:
URL: https://github.com/apache/pulsar/pull/20049#issuecomment-1501497950
/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
poorbarcode commented on PR #13080:
URL: https://github.com/apache/pulsar/pull/13080#issuecomment-1501519358
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
github-actions[bot] commented on PR #13080:
URL: https://github.com/apache/pulsar/pull/13080#issuecomment-1501519622
@rdhabalia Please add the following content to your PR description and
select a checkbox:
```
- [ ] `doc`
- [ ] `doc-required`
- [ ] `doc-not-needed`
- [ ]
poorbarcode commented on PR #19465:
URL: https://github.com/apache/pulsar/pull/19465#issuecomment-1501541527
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
Anonymitaet commented on PR #19888:
URL: https://github.com/apache/pulsar/pull/19888#issuecomment-1501545677
Hi @michaeljmarshall thanks for bringing this feature!
I see this PR is labeled with doc-required, for these new configurations, do
we need to add some instructions and usage
poorbarcode commented on PR #14560:
URL: https://github.com/apache/pulsar/pull/14560#issuecomment-1501550051
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #14286:
URL: https://github.com/apache/pulsar/pull/14286#issuecomment-1501569100
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #15628:
URL: https://github.com/apache/pulsar/pull/15628#issuecomment-1501575654
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on code in PR #15628:
URL: https://github.com/apache/pulsar/pull/15628#discussion_r1161560793
##
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentMessageExpiryMonitor.java:
##
@@ -77,8 +77,10 @@ public boolean
poorbarcode commented on PR #17524:
URL: https://github.com/apache/pulsar/pull/17524#issuecomment-1501577805
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
sindhushreem commented on issue #20013:
URL: https://github.com/apache/pulsar/issues/20013#issuecomment-1501585368
@lhotari
We do have snake yaml direct dependency in buildtools pom.xml .
--
This is an automated message from the Apache Git Service.
To respond to the message, please
TakaHiR07 opened a new pull request, #20053:
URL: https://github.com/apache/pulsar/pull/20053
### Motivation
When updateTxnStatus concurrently, there is a case only one succeed, others
may throw exception.
If the newStatus == txnMeta.status(), it should also succeed.
tisonkun commented on issue #19456:
URL: https://github.com/apache/pulsar/issues/19456#issuecomment-1501490520
@niharrathod are you working on this issue? I can assign it to you if you're
still willing to continue :)
--
This is an automated message from the Apache Git Service.
To respond
poorbarcode commented on PR #19571:
URL: https://github.com/apache/pulsar/pull/19571#issuecomment-1501531407
/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
poorbarcode commented on PR #18007:
URL: https://github.com/apache/pulsar/pull/18007#issuecomment-1501544929
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #16792:
URL: https://github.com/apache/pulsar/pull/16792#issuecomment-1501548259
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #15594:
URL: https://github.com/apache/pulsar/pull/15594#issuecomment-1501547582
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
BewareMyPower commented on code in PR #1002:
URL: https://github.com/apache/pulsar-client-go/pull/1002#discussion_r1161505707
##
pulsar/producer_partition.go:
##
@@ -1067,6 +1108,20 @@ func (p *partitionProducer) SendAsync(ctx
context.Context, msg *ProducerMessage,
func (p
poorbarcode commented on PR #13298:
URL: https://github.com/apache/pulsar/pull/13298#issuecomment-1501567586
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #15494:
URL: https://github.com/apache/pulsar/pull/15494#issuecomment-1501571068
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #17485:
URL: https://github.com/apache/pulsar/pull/17485#issuecomment-1501578534
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on code in PR #20050:
URL: https://github.com/apache/pulsar/pull/20050#discussion_r1161502480
##
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/MetaStoreImpl.java:
##
@@ -302,16 +302,17 @@ public void asyncRemoveCursor(String ledgerName,
poorbarcode commented on PR #15852:
URL: https://github.com/apache/pulsar/pull/15852#issuecomment-1501515145
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19498:
URL: https://github.com/apache/pulsar/pull/19498#issuecomment-1501538346
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19344:
URL: https://github.com/apache/pulsar/pull/19344#issuecomment-1501543710
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #17846:
URL: https://github.com/apache/pulsar/pull/17846#issuecomment-1501546499
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #15965:
URL: https://github.com/apache/pulsar/pull/15965#issuecomment-1501580835
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
tisonkun commented on issue #20041:
URL: https://github.com/apache/pulsar/issues/20041#issuecomment-1501598862
The error logs said "IllegalArgumentException", maybe you can change:
```
pulsar.nar-extraction-directory=System.getProperty("java.io.tmpdir")
```
to
```
gaoran10 opened a new pull request, #1006:
URL: https://github.com/apache/pulsar-client-go/pull/1006
### Motivation
The pulsar-client-go supports ProtoSchema based on the Avro format, users
must provide the schema definition string when using struct schema(JSON, Avro,
Proto), it's
tisonkun opened a new pull request, #1007:
URL: https://github.com/apache/pulsar-client-go/pull/1007
This closes #1005.
### Modifications
*Describe the modifications you've done.*
### Verifying this change
- [x] Make sure that the change passes the CI checks.
tisonkun commented on PR #1004:
URL:
https://github.com/apache/pulsar-client-go/pull/1004#issuecomment-1501827529
I'm taking over #1005 and open a new PR at #1007. Welcome to give a review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
tisonkun closed pull request #1004: test: test ci lint
URL: https://github.com/apache/pulsar-client-go/pull/1004
--
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
codecov-commenter commented on PR #20009:
URL: https://github.com/apache/pulsar/pull/20009#issuecomment-1501838882
##
[Codecov](https://codecov.io/gh/apache/pulsar/pull/20009?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
tisonkun commented on issue #19905:
URL: https://github.com/apache/pulsar/issues/19905#issuecomment-1501840947
@lhotari any feedback or new instance? I don't find a similar case now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
mattisonchao commented on code in PR #20030:
URL: https://github.com/apache/pulsar/pull/20030#discussion_r1161752440
##
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderBasic.java:
##
@@ -104,10 +105,11 @@ public String
mattisonchao commented on PR #20030:
URL: https://github.com/apache/pulsar/pull/20030#issuecomment-1501868653
@michaeljmarshall would you mind to taking a look here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
mattisonchao commented on PR #20050:
URL: https://github.com/apache/pulsar/pull/20050#issuecomment-1501869814
+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 above to go to the specific comment.
To
tisonkun commented on issue #19785:
URL: https://github.com/apache/pulsar/issues/19785#issuecomment-1501875429
This should be expected. The class `org.conscrypt.Conscrypt` is not shaded,
and it's logically different from the Netty's one.
Please check if it's a non-fatal exception and
GitHub user jeantil edited a discussion: pulsar admin client in
pulsar-client-all logs a ClassNotFoundException at startup
### Search before asking
- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and
found nothing similar.
### Version
2.11.0
### Minimal
tisonkun closed issue #19785: [Bug] pulsar admin client in pulsar-client-all
logs a ClassNotFoundException at startup
URL: https://github.com/apache/pulsar/issues/19785
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
GitHub user tisonkun added a comment to the discussion: [Bug] pulsar admin
client in pulsar-client-all logs a ClassNotFoundException at startup
This should be expected. The class `org.conscrypt.Conscrypt` is not shaded, and
it's logically different from the Netty's one.
Please check if it's
tisonkun commented on issue #20013:
URL: https://github.com/apache/pulsar/issues/20013#issuecomment-1501879975
> in buildtools pom.xml
Yes. It's unnormal. We should remove it or move it to the
`dependencyManagement` group.
> and parent pom.xml
No. It's only in the
tisonkun commented on issue #20013:
URL: https://github.com/apache/pulsar/issues/20013#issuecomment-1501881962
> 2.15 hasn't been released yet, it's currently 2.15.0-rc2 .
> Perhaps we should consider Jackson 2.15 for Pulsar 3.0 ?
Yeah. I noticed that jackson 2.15 is unreleased
shibd commented on issue #18616:
URL: https://github.com/apache/pulsar/issues/18616#issuecomment-1501882347
> @BewareMyPower : serializing and deserializing is expensive and on top of
that having different APIs for different use cases is creating a really bad
experience for users, and I
shibd commented on code in PR #20040:
URL: https://github.com/apache/pulsar/pull/20040#discussion_r1161756544
##
pulsar-broker/src/test/java/org/apache/pulsar/client/impl/TopicsConsumerImplTest.java:
##
@@ -1170,6 +1193,20 @@ public void testGetLastMessageId() throws Exception
shibd commented on PR #1007:
URL:
https://github.com/apache/pulsar-client-go/pull/1007#issuecomment-1501891144
@tisonkun Please help explain why?
--
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
codecov-commenter commented on PR #19815:
URL: https://github.com/apache/pulsar/pull/19815#issuecomment-1501992460
##
[Codecov](https://codecov.io/gh/apache/pulsar/pull/19815?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
poorbarcode commented on PR #19432:
URL: https://github.com/apache/pulsar/pull/19432#issuecomment-1502057131
/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
michaeljmarshall commented on code in PR #20026:
URL: https://github.com/apache/pulsar/pull/20026#discussion_r1162017346
##
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/HashRangeExclusiveStickyKeyConsumerSelector.java:
##
@@ -52,8 +54,23 @@ public
This is an automated email from the ASF dual-hosted git repository.
mmarshall 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 11751b7da23 [feat] PIP-257: Add
michaeljmarshall merged PR #19849:
URL: https://github.com/apache/pulsar/pull/19849
--
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:
poorbarcode commented on PR #17964:
URL: https://github.com/apache/pulsar/pull/17964#issuecomment-1502051004
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
poorbarcode commented on PR #19944:
URL: https://github.com/apache/pulsar/pull/19944#issuecomment-1502062686
Since we will start the RC version of `3.0.0` on `2023-04-11`, I will change
the label/milestone of PR who have not been merged.
- The PR of type `feature` is deferred to `3.1.0`
GitHub user svabra added a comment to the discussion: [Bug] Failed to pull-up
docker-compose for broker, bookie, zk and pulsar-sql
@tisonkun thanks for having a look. Yes, as Java Developer I know. But, think
of it not for me, but for people new to Pulsar and PulsarSQL. Why are we not
chrisjohnson00 commented on PR #356:
URL:
https://github.com/apache/pulsar-helm-chart/pull/356#issuecomment-1502215114
> > Oh, I hadn't noticed that both the service and the deployment shared
these values. Wouldn't this make it a breaking change since the service ports
would change? I'd
1 - 100 of 257 matches
Mail list logo