[GitHub] [pulsar] abhilashmandaliya added a comment to the discussion: Bookkeeper startup error: NoNode for /ledgers/LAYOUT

2023-03-21 Thread GitBox
GitHub user abhilashmandaliya added a comment to the discussion: Bookkeeper startup error: NoNode for /ledgers/LAYOUT Got some help from the community over slack. They pointed me to the cluster metadata initialization

[GitHub] [pulsar] poorbarcode commented on pull request #19841: [fix] [broker] fix infinite HTTP call getSubscriptions caused by wrong topicName

2023-03-21 Thread via GitHub
poorbarcode commented on PR #19841: URL: https://github.com/apache/pulsar/pull/19841#issuecomment-1477332086 Hi @wangjialing218 > This fix could solve https://github.com/apache/pulsar/issues/18149 Yes, I've seen the issue, and I'm sure it's the same one -- This is an

[GitHub] [pulsar] Demogorgon314 commented on a diff in pull request #19851: [improve][broker] PIP-192: Support delete namespace bundle admin API

2023-03-21 Thread via GitHub
Demogorgon314 commented on code in PR #19851: URL: https://github.com/apache/pulsar/pull/19851#discussion_r1144182109 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -1445,23 +1451,44 @@ public PulsarClientImpl

[GitHub] [pulsar] lordcheng10 opened a new pull request, #19890: [fix][broker] Fix overloadedBroker and underloadedBroker to select the wrong broker for UniformLoadShedder

2023-03-21 Thread via GitHub
lordcheng10 opened a new pull request, #19890: URL: https://github.com/apache/pulsar/pull/19890 ### Motivation The configuration is as follows: ``` loadBalancerMsgThroughputMultiplierDifferenceShedderThreshold=0 loadBalancerMsgRateDifferenceShedderThreshold=50 ``` I

[GitHub] [pulsar] Demogorgon314 commented on a diff in pull request #19851: [improve][broker] PIP-192: Support delete namespace bundle admin API

2023-03-21 Thread via GitHub
Demogorgon314 commented on code in PR #19851: URL: https://github.com/apache/pulsar/pull/19851#discussion_r1144182522 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -1165,8 +1165,14 @@ public CompletableFuture

[GitHub] [pulsar-client-python] RobertIndie closed issue #107: [feature request] Support Dead Letter Topic

2023-03-21 Thread via GitHub
RobertIndie closed issue #107: [feature request] Support Dead Letter Topic URL: https://github.com/apache/pulsar-client-python/issues/107 -- 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

[GitHub] [pulsar-client-python] RobertIndie commented on issue #107: [feature request] Support Dead Letter Topic

2023-03-21 Thread via GitHub
RobertIndie commented on issue #107: URL: https://github.com/apache/pulsar-client-python/issues/107#issuecomment-1478882308 This is a duplicate of https://github.com/apache/pulsar-client-python/issues/98 -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-site] momo-jun opened a new pull request, #486: [improve][doc] Add schema support for python functions

2023-03-21 Thread via GitHub
momo-jun opened a new pull request, #486: URL: https://github.com/apache/pulsar-site/pull/486 ### Modification Add docs for https://github.com/apache/pulsar/pull/18432 1. Add a table to show the supportability of schema types for language-specific function instances 2. Add a

[GitHub] [pulsar] 315157973 commented on a diff in pull request #19874: [fix][client] Fix NPE when acknowledging multiple messages

2023-03-21 Thread via GitHub
315157973 commented on code in PR #19874: URL: https://github.com/apache/pulsar/pull/19874#discussion_r1144268868 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImpl.java: ## @@ -496,8 +496,13 @@ protected CompletableFuture doAcknowledge(List

[GitHub] [pulsar] poorbarcode commented on pull request #19841: [fix] [broker] fix infinite HTTP call getSubscriptions caused by wrong topicName

2023-03-21 Thread via GitHub
poorbarcode commented on PR #19841: URL: https://github.com/apache/pulsar/pull/19841#issuecomment-1478851084 /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

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #19851: [improve][broker] PIP-192: Support delete namespace bundle admin API

2023-03-21 Thread via GitHub
Technoboy- commented on code in PR #19851: URL: https://github.com/apache/pulsar/pull/19851#discussion_r1144181575 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -1165,8 +1165,14 @@ public CompletableFuture

[GitHub] [pulsar] poorbarcode closed pull request #19825: [fix] [ml] make the result of delete cursor is success if cursor is deleted

2023-03-21 Thread via GitHub
poorbarcode closed pull request #19825: [fix] [ml] make the result of delete cursor is success if cursor is deleted URL: https://github.com/apache/pulsar/pull/19825 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #19851: [improve][broker] PIP-192: Support delete namespace bundle admin API

2023-03-21 Thread via GitHub
Technoboy- commented on code in PR #19851: URL: https://github.com/apache/pulsar/pull/19851#discussion_r1144186547 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -1165,8 +1165,14 @@ public CompletableFuture

[GitHub] [pulsar] NiuBlibing opened a new issue, #19886: Metrics: seperate topic-partion into topic and partion

2023-03-21 Thread via GitHub
NiuBlibing opened a new issue, #19886: URL: https://github.com/apache/pulsar/issues/19886 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Motivation `pulsar_msg_backlog` is use tag

[GitHub] [pulsar] tisonkun commented on issue #19877: [Doc] Add API doc for the C# client

2023-03-21 Thread via GitHub
tisonkun commented on issue #19877: URL: https://github.com/apache/pulsar/issues/19877#issuecomment-1478866391 I suggest we pull dotpulsar authors @entvex @blankensteiner for comments and helps :) I'm unfamiliar with C# API doc generation yet. -- This is an automated message from

[GitHub] [pulsar] poorbarcode commented on a diff in pull request #19014: [fix] [doc] Fix outdated java-doc of rate limiter

2023-03-21 Thread via GitHub
poorbarcode commented on code in PR #19014: URL: https://github.com/apache/pulsar/pull/19014#discussion_r1057408809 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/DispatchRateLimiter.java: ## @@ -99,10 +99,10 @@ public long

[GitHub] [pulsar] lordcheng10 opened a new pull request, #19887: [fix][broker] Fix the loss of bundle stats data reported to zookeeper, when the updateStats method is executed

2023-03-21 Thread via GitHub
lordcheng10 opened a new pull request, #19887: URL: https://github.com/apache/pulsar/pull/19887 ### Motivation When the updateStats method is executed periodically, bundleStats will be cleaned up; At this time, call the getBundleStats method to report the bundle stats data to

[GitHub] [pulsar] lordcheng10 opened a new pull request, #19889: [fix][broker] Fix BrokerData#timeAverageData update is thread safe

2023-03-21 Thread via GitHub
lordcheng10 opened a new pull request, #19889: URL: https://github.com/apache/pulsar/pull/19889 ### Motivation In the updateBundleData method, when the following code is executed:

[GitHub] [pulsar-site] momo-jun merged pull request #461: [feat][doc] Add new configs `skipToLatest` and `compressionType`

2023-03-21 Thread via GitHub
momo-jun merged PR #461: URL: https://github.com/apache/pulsar-site/pull/461 -- 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:

[GitHub] [pulsar] michaeljmarshall opened a new pull request, #19888: [feat][fn] PIP-257: Support mounting k8s ServiceAccount for OIDC auth

2023-03-21 Thread via GitHub
michaeljmarshall opened a new pull request, #19888: URL: https://github.com/apache/pulsar/pull/19888 PIP: #19771 ### Motivation In order to make OIDC work with functions, we must give them a way to authenticate with the broker using tokens that are able to be validated by an

[GitHub] [pulsar] Anonymitaet commented on issue #19877: [Doc] Add API doc for the C# client

2023-03-21 Thread via GitHub
Anonymitaet commented on issue #19877: URL: https://github.com/apache/pulsar/issues/19877#issuecomment-1478858980 I've discussed with @RobertIndie, to resolve this issue, we need to finish the following tasks: 1. Generate the API reference doc. We want to generate C#

[pulsar-site] branch main updated: [feat][doc] Add new configs `skipToLatest` and `compressionType` (#461)

2023-03-21 Thread junma
This is an automated email from the ASF dual-hosted git repository. junma 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 618cfca710c [feat][doc] Add new configs

[GitHub] [pulsar-client-go] crossoverJie commented on issue #919: The behavior of `Seek` is different with Java Client

2023-03-21 Thread via GitHub
crossoverJie commented on issue #919: URL: https://github.com/apache/pulsar-client-go/issues/919#issuecomment-1478936014 I am interested in implementing this, please assign it to me, thanks. @RobertIndie -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] heesung-sn commented on a diff in pull request #19851: [improve][broker] PIP-192: Support delete namespace bundle admin API

2023-03-21 Thread via GitHub
heesung-sn commented on code in PR #19851: URL: https://github.com/apache/pulsar/pull/19851#discussion_r1144258938 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -1445,23 +1451,44 @@ public PulsarClientImpl

[GitHub] [pulsar-site] tisonkun opened a new pull request, #485: Add issue triaging doc

2023-03-21 Thread via GitHub
tisonkun opened a new pull request, #485: URL: https://github.com/apache/pulsar-site/pull/485 ### Documentation - [x] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` - [ ] `doc-complete` -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] Anonymitaet commented on issue #19876: [Doc] Add API doc for NodeJS client

2023-03-21 Thread via GitHub
Anonymitaet commented on issue #19876: URL: https://github.com/apache/pulsar/issues/19876#issuecomment-1478859322 Similar to https://github.com/apache/pulsar/issues/19877#issuecomment-1478858980 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar-site] momo-jun commented on pull request #486: [improve][doc] Add schema support for python functions

2023-03-21 Thread via GitHub
momo-jun commented on PR #486: URL: https://github.com/apache/pulsar-site/pull/486#issuecomment-147058 Hi @jiangpengcheng, @freeznet , can you pls review the doc PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar-client-go] liangyepianzhou commented on a diff in pull request #984: [feat][txn]Implement transactionImpl

2023-03-21 Thread via GitHub
liangyepianzhou commented on code in PR #984: URL: https://github.com/apache/pulsar-client-go/pull/984#discussion_r1142975867 ## pulsar/transaction_impl.go: ## @@ -0,0 +1,232 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

[GitHub] [pulsar] Anonymitaet commented on issue #14561: [Doc] [pulsar-io] Missing documentation for the BatchSource interface for Pulsar IO

2023-03-21 Thread via GitHub
Anonymitaet commented on issue #14561: URL: https://github.com/apache/pulsar/issues/14561#issuecomment-1477404433 @hsaputra adding a "how-to" doc for each interface is an unfeasible solution (e.g., adding a tutorial for every interface to https://pulsar.apache.org/docs/en/io-develop/)

[GitHub] [pulsar-dotpulsar] RobertIndie closed pull request #58: Basic message encryption

2023-03-21 Thread via GitHub
RobertIndie closed pull request #58: Basic message encryption URL: https://github.com/apache/pulsar-dotpulsar/pull/58 -- 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

[pulsar] branch master updated: [fix][doc] The `messageIds` in `onNegativeAcksSend` shouldn't be null (#19852)

2023-03-21 Thread zike
This is an automated email from the ASF dual-hosted git repository. zike 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 1e44ba179bf [fix][doc] The `messageIds` in

[GitHub] [pulsar] RobertIndie merged pull request #19852: [fix][doc] The `messageIds` in `onNegativeAcksSend` shouldn't be null

2023-03-21 Thread via GitHub
RobertIndie merged PR #19852: URL: https://github.com/apache/pulsar/pull/19852 -- 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:

[GitHub] [pulsar-site] tisonkun merged pull request #478: [improve][misc] trim all whitespaces

2023-03-21 Thread via GitHub
tisonkun merged PR #478: URL: https://github.com/apache/pulsar-site/pull/478 -- 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:

[pulsar-client-node] branch master updated: [Fix] Wrap c++ exception when creating client. (#307)

2023-03-21 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git The following commit(s) were added to refs/heads/master by this push: new 64e44c1 [Fix] Wrap c++ exception

[GitHub] [pulsar-client-cpp] shibd closed issue #89: [C++ Client] New consumer type: TableView

2023-03-21 Thread via GitHub
shibd closed issue #89: [C++ Client] New consumer type: TableView URL: https://github.com/apache/pulsar-client-cpp/issues/89 -- 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

[GitHub] [pulsar] asafm commented on pull request #19832: [fix][misc] Change PIP issue template

2023-03-21 Thread via GitHub
asafm commented on PR #19832: URL: https://github.com/apache/pulsar/pull/19832#issuecomment-1477611911 It's better. Can you provide an example of a feature in pulsar which is dealing correctly with multi-tenancy? Once I have a good example and a bad example, it will be easier to

[pulsar] branch branch-2.10 updated: [improve][admin][branch-2.10] Unset namespace policy to improve deleting namespace (#17033) (#19865)

2023-03-21 Thread xiangying
This is an automated email from the ASF dual-hosted git repository. xiangying pushed a commit to branch branch-2.10 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.10 by this push: new 70b86175f97

[pulsar-client-go] branch xiangying/txn/transaction_impl updated (42dde3c -> 0359ac1)

2023-03-21 Thread xiangying
This is an automated email from the ASF dual-hosted git repository. xiangying pushed a change to branch xiangying/txn/transaction_impl in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git from 42dde3c Add API doc add 0359ac1 double check No new revisions were added

[GitHub] [pulsar] liangyepianzhou merged pull request #19865: [improve][admin][branch-2.10] Unset namespace policy to improve deleting namespace (#17033)

2023-03-21 Thread via GitHub
liangyepianzhou merged PR #19865: URL: https://github.com/apache/pulsar/pull/19865 -- 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:

[GitHub] [pulsar] Anonymitaet commented on pull request #18929: [improve][fn] Support processingGuarantees "EFFECTIVELY_ONCE" in python function

2023-03-21 Thread via GitHub
Anonymitaet commented on PR #18929: URL: https://github.com/apache/pulsar/pull/18929#issuecomment-1477383424 Hi @tpiperatgod @freeznet @nlu90 this PR is labeled with `doc-required`. I check the relevant docs and have some questions below. Can you please double-check? Thank you! 1.

[pulsar] annotated tag v2.10.4-candidate-3 updated (e4898ac8eb3 -> ec25a0f8cbd)

2023-03-21 Thread xiangying
This is an automated email from the ASF dual-hosted git repository. xiangying pushed a change to annotated tag v2.10.4-candidate-3 in repository https://gitbox.apache.org/repos/asf/pulsar.git *** WARNING: tag v2.10.4-candidate-3 was modified! *** from e4898ac8eb3 (commit) to

[GitHub] [pulsar-dotpulsar] RobertIndie closed pull request #57: Batching when producing

2023-03-21 Thread via GitHub
RobertIndie closed pull request #57: Batching when producing URL: https://github.com/apache/pulsar-dotpulsar/pull/57 -- 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

[GitHub] [pulsar-client-node] shibd commented on a diff in pull request #307: [Fix] Wrap c++ exception when creating client.

2023-03-21 Thread via GitHub
shibd commented on code in PR #307: URL: https://github.com/apache/pulsar-client-node/pull/307#discussion_r1143002369 ## tests/client.test.js: ## @@ -0,0 +1,46 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements.

[GitHub] [pulsar-dotpulsar] RobertIndie closed pull request #49: partitioned consumer

2023-03-21 Thread via GitHub
RobertIndie closed pull request #49: partitioned consumer URL: https://github.com/apache/pulsar-dotpulsar/pull/49 -- 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

[GitHub] [pulsar-dotpulsar] RobertIndie closed pull request #81: Partitioned Consumer

2023-03-21 Thread via GitHub
RobertIndie closed pull request #81: Partitioned Consumer URL: https://github.com/apache/pulsar-dotpulsar/pull/81 -- 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

[GitHub] [pulsar] RobertIndie opened a new issue, #19876: [Doc] Add API doc for NodeJS client

2023-03-21 Thread via GitHub
RobertIndie opened a new issue, #19876: URL: https://github.com/apache/pulsar/issues/19876 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### What issue do you find in Pulsar docs? There

[GitHub] [pulsar] Technoboy- closed pull request #19875: [fix][broker] Fix NPE when update topic policy.

2023-03-21 Thread via GitHub
Technoboy- closed pull request #19875: [fix][broker] Fix NPE when update topic policy. URL: https://github.com/apache/pulsar/pull/19875 -- 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

[GitHub] [pulsar] momo-jun closed issue #15035: [Doc] Java client examples for protobuf native schema

2023-03-21 Thread via GitHub
momo-jun closed issue #15035: [Doc] Java client examples for protobuf native schema URL: https://github.com/apache/pulsar/issues/15035 -- 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

[GitHub] [pulsar] momo-jun commented on issue #15035: [Doc] Java client examples for protobuf native schema

2023-03-21 Thread via GitHub
momo-jun commented on issue #15035: URL: https://github.com/apache/pulsar/issues/15035#issuecomment-1477432867 The required code example has been added [here](https://pulsar.apache.org/docs/2.11.x/schema-get-started/#protobufnative). -- This is an automated message from the Apache Git

[pulsar-client-cpp] branch main updated: Fix possible crash caused by MessageId::getTopicName (#225)

2023-03-21 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-client-cpp.git The following commit(s) were added to refs/heads/main by this push: new ba52218 Fix possible crash caused by

[GitHub] [pulsar] poorbarcode opened a new pull request, #19879: [fix] [admin] Make response code to 400 instead of 500 when delete topic fails due to enabled geo-replication

2023-03-21 Thread via GitHub
poorbarcode opened a new pull request, #19879: URL: https://github.com/apache/pulsar/pull/19879 ### Motivation As expected, If geo-replication is enabled, a topic cannot be deleted. However deleting that topic returns a 500, and no further info. ``` 2023-03-21T18:09:57,909 -

[GitHub] [pulsar] tisonkun commented on issue #19429: Make Java Client Authentication Asynchronous

2023-03-21 Thread via GitHub
tisonkun commented on issue #19429: URL: https://github.com/apache/pulsar/issues/19429#issuecomment-1477590740 @michaeljmarshall shall we make a PIP for this enhancement? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] asafm commented on pull request #19832: [fix][misc] Change PIP issue template

2023-03-21 Thread via GitHub
asafm commented on PR #19832: URL: https://github.com/apache/pulsar/pull/19832#issuecomment-1477614070 Regarding stateStore, you're basically saying they didn't take into consideration the security aspects of multi-tenancy, since that interface allows consuming any state by any tenant.

[GitHub] [pulsar-site] momo-jun commented on a diff in pull request #461: [feat][doc] Add new configs `skipToLatest` and `compressionType`

2023-03-21 Thread via GitHub
momo-jun commented on code in PR #461: URL: https://github.com/apache/pulsar-site/pull/461#discussion_r1142943256 ## docs/functions-cli.md: ## @@ -63,7 +63,7 @@ You can configure a function by using a predefined YAML file. The following tabl | maxPendingAsyncRequests | Int

[GitHub] [pulsar] startjava opened a new issue, #19872: [Doc] the "default configuration file" link no page !

2023-03-21 Thread via GitHub
startjava opened a new issue, #19872: URL: https://github.com/apache/pulsar/issues/19872 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### What issue do you find in Pulsar docs?

[GitHub] [pulsar-site] tisonkun opened a new pull request, #478: [improve][misc] trim all whitespaces

2023-03-21 Thread via GitHub
tisonkun opened a new pull request, #478: URL: https://github.com/apache/pulsar-site/pull/478 ### Documentation - [ ] `doc` - [ ] `doc-required` - [x] `doc-not-needed` - [ ] `doc-complete` -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] BewareMyPower opened a new pull request, #19874: [fix][client] Fix NPE when acknowledging multiple messages

2023-03-21 Thread via GitHub
BewareMyPower opened a new pull request, #19874: URL: https://github.com/apache/pulsar/pull/19874 ### Motivation For a multi-topics consumer, when it acknowledges a single message, it will first find the owner topic from its message ID. If the owner topic is not subscribed by the

[GitHub] [pulsar] RobertIndie opened a new issue, #19877: [Doc] Add API doc for the C# client

2023-03-21 Thread via GitHub
RobertIndie opened a new issue, #19877: URL: https://github.com/apache/pulsar/issues/19877 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### What issue do you find in Pulsar docs? There

[GitHub] [pulsar] tisonkun closed issue #19872: [Doc] the "default configuration file" link no page !

2023-03-21 Thread via GitHub
tisonkun closed issue #19872: [Doc] the "default configuration file" link no page ! URL: https://github.com/apache/pulsar/issues/19872 -- 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

[GitHub] [pulsar] tisonkun commented on issue #19872: [Doc] the "default configuration file" link no page !

2023-03-21 Thread via GitHub
tisonkun commented on issue #19872: URL: https://github.com/apache/pulsar/issues/19872#issuecomment-1477350966 It links to https://github.com/apache/pulsar/blob/e6b12c64b043903eb5ff2dc5186fe8030f157cfc/conf/standalone.conf which is a valid page. Please provide a screen recording if

[GitHub] [pulsar] tisonkun commented on a diff in pull request #19871: [fix][client] Fix typo in ConsumerStats

2023-03-21 Thread via GitHub
tisonkun commented on code in PR #19871: URL: https://github.com/apache/pulsar/pull/19871#discussion_r1142947002 ## pulsar-client-api/src/main/java/org/apache/pulsar/client/api/ConsumerStats.java: ## @@ -92,7 +92,7 @@ public interface ConsumerStats extends Serializable {

[GitHub] [pulsar] tisonkun commented on a diff in pull request #19871: [fix][client] Fix typo in ConsumerStats

2023-03-21 Thread via GitHub
tisonkun commented on code in PR #19871: URL: https://github.com/apache/pulsar/pull/19871#discussion_r1142947002 ## pulsar-client-api/src/main/java/org/apache/pulsar/client/api/ConsumerStats.java: ## @@ -92,7 +92,7 @@ public interface ConsumerStats extends Serializable {

[pulsar] 01/01: cs

2023-03-21 Thread xiangying
This is an automated email from the ASF dual-hosted git repository. xiangying pushed a commit to branch ne in repository https://gitbox.apache.org/repos/asf/pulsar.git commit e4898ac8eb37f698f29aa21e40a3abdda5489d45 Author: xiangying <1984997...@qq.com> AuthorDate: Tue Mar 21 15:47:02 2023 +0800

[pulsar] branch ne created (now e4898ac8eb3)

2023-03-21 Thread xiangying
This is an automated email from the ASF dual-hosted git repository. xiangying pushed a change to branch ne in repository https://gitbox.apache.org/repos/asf/pulsar.git at e4898ac8eb3 cs This branch includes the following new commits: new e4898ac8eb3 cs The 1 revisions listed above

[GitHub] [pulsar] BewareMyPower commented on pull request #19874: [fix][client] Fix NPE when acknowledging multiple messages

2023-03-21 Thread via GitHub
BewareMyPower commented on PR #19874: URL: https://github.com/apache/pulsar/pull/19874#issuecomment-1477421964 I decide to ignore these invalid message IDs because in real world, it usually happens when these messages are not received by the consumer or the topic of them has been deleted

[GitHub] [pulsar-client-cpp] BewareMyPower opened a new pull request, #225: Fix possible crash caused by MessageId::getTopicName

2023-03-21 Thread via GitHub
BewareMyPower opened a new pull request, #225: URL: https://github.com/apache/pulsar-client-cpp/pull/225 ### Motivation Currently if a `MessageId` does not have a topic name, the `getTopicName` method will dereference a null pointer, which leads to a crash. This case usually happens

[GitHub] [pulsar-client-node] shibd merged pull request #307: [Fix] Wrap c++ exception when creating client.

2023-03-21 Thread via GitHub
shibd merged PR #307: URL: https://github.com/apache/pulsar-client-node/pull/307 -- 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:

[GitHub] [pulsar-client-node] shibd closed issue #303: libc++abi: terminating with uncaught exception of type std::invalid_argument: The scheme part is missing:

2023-03-21 Thread via GitHub
shibd closed issue #303: libc++abi: terminating with uncaught exception of type std::invalid_argument: The scheme part is missing: URL: https://github.com/apache/pulsar-client-node/issues/303 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar-site] RobertIndie opened a new pull request, #479: [improve][doc] Add go client configs

2023-03-21 Thread via GitHub
RobertIndie opened a new pull request, #479: URL: https://github.com/apache/pulsar-site/pull/479 Add go client configs. https://user-images.githubusercontent.com/16974619/226565962-a97e9d14-3bd3-41d8-aae0-ae77d2e59c23.png;> ### Documentation - [x] `doc` -

[GitHub] [pulsar] RobertIndie opened a new issue, #19878: [Doc] 404 links in the java client config doc

2023-03-21 Thread via GitHub
RobertIndie opened a new issue, #19878: URL: https://github.com/apache/pulsar/issues/19878 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### What issue do you find in Pulsar docs? There

[GitHub] [pulsar-site] tisonkun opened a new pull request, #480: [fix][doc] Links and tables

2023-03-21 Thread via GitHub
tisonkun opened a new pull request, #480: URL: https://github.com/apache/pulsar-site/pull/480 This fixes https://github.com/apache/pulsar/issues/19878. ### Documentation - [x] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` - [ ] `doc-complete` --

[GitHub] [pulsar-client-cpp] BewareMyPower merged pull request #225: Fix possible crash caused by MessageId::getTopicName

2023-03-21 Thread via GitHub
BewareMyPower merged PR #225: URL: https://github.com/apache/pulsar-client-cpp/pull/225 -- 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:

[GitHub] [pulsar] tisonkun closed issue #19878: [Doc] 404 links in the java client config doc

2023-03-21 Thread via GitHub
tisonkun closed issue #19878: [Doc] 404 links in the java client config doc URL: https://github.com/apache/pulsar/issues/19878 -- 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.

[GitHub] [pulsar-site] tisonkun merged pull request #480: [fix][doc] Links and tables

2023-03-21 Thread via GitHub
tisonkun merged PR #480: URL: https://github.com/apache/pulsar-site/pull/480 -- 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:

[GitHub] [pulsar-site] tisonkun opened a new pull request, #481: [feat] Add canonical link for SEO

2023-03-21 Thread via GitHub
tisonkun opened a new pull request, #481: URL: https://github.com/apache/pulsar-site/pull/481 This closes https://github.com/apache/pulsar/issues/18190. ### Documentation - [ ] `doc` - [ ] `doc-required` - [x] `doc-not-needed` - [ ] `doc-complete` --

[GitHub] [pulsar-site] tisonkun merged pull request #477: Replace commerical page with tech docs

2023-03-21 Thread via GitHub
tisonkun merged PR #477: URL: https://github.com/apache/pulsar-site/pull/477 -- 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:

[GitHub] [pulsar] Technoboy- opened a new pull request, #19875: [fix][broker] Fix NPE when update topic policy.

2023-03-21 Thread via GitHub
Technoboy- opened a new pull request, #19875: URL: https://github.com/apache/pulsar/pull/19875 ### Motivation ``` Failed to create consumer: consumerId=30, Cannot invoke "java.util.Map.get(Object)" because "this.subscriptionPolicies" is null ``` ### Documentation

[GitHub] [pulsar-client-go] fracasula opened a new issue, #995: When a new release?

2023-03-21 Thread via GitHub
fracasula opened a new issue, #995: URL: https://github.com/apache/pulsar-client-go/issues/995 Hey guys, when are you planning a new release? I see that v0.9.0 is from October 2022, quite a while ago considering all fixes that got through since then:

[GitHub] [pulsar-client-cpp] BewareMyPower commented on pull request #196: [TableView-2] Implement all interfaces.

2023-03-21 Thread via GitHub
BewareMyPower commented on PR #196: URL: https://github.com/apache/pulsar-client-cpp/pull/196#issuecomment-1477524660 Should we close https://github.com/apache/pulsar-client-cpp/issues/89 now? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar-client-cpp] BewareMyPower closed issue #50: [Feature] Supports producers to automatically obtain partitions

2023-03-21 Thread via GitHub
BewareMyPower closed issue #50: [Feature] Supports producers to automatically obtain partitions URL: https://github.com/apache/pulsar-client-cpp/issues/50 -- 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

[GitHub] [pulsar] tisonkun commented on issue #19878: [Doc] 404 links in the java client config doc

2023-03-21 Thread via GitHub
tisonkun commented on issue #19878: URL: https://github.com/apache/pulsar/issues/19878#issuecomment-1477534304 I'm handling this now. -- 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

[GitHub] [pulsar] urfreespace commented on issue #18190: [Bug] It's hard to find latest docs using search engines like Google

2023-03-21 Thread via GitHub
urfreespace commented on issue #18190: URL: https://github.com/apache/pulsar/issues/18190#issuecomment-1477613803 > To summarize the possible solutions based on my understanding: > > 1. **Preferred**: Update the [logic of the existing global canonical URL

[GitHub] [pulsar] tisonkun commented on issue #18190: [Bug] It's hard to find latest docs using search engines like Google

2023-03-21 Thread via GitHub
tisonkun commented on issue #18190: URL: https://github.com/apache/pulsar/issues/18190#issuecomment-1477624092 @urfreespace perhaps you can eject something like `DocHead` by `yarn swizzle`. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] tisonkun commented on issue #18190: [Bug] It's hard to find latest docs using search engines like Google

2023-03-21 Thread via GitHub
tisonkun commented on issue #18190: URL: https://github.com/apache/pulsar/issues/18190#issuecomment-1477653824 Here is a patch https://github.com/apache/pulsar-site/pull/481 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar-site] tisonkun commented on pull request #481: [feat] Add canonical link for SEO

2023-03-21 Thread via GitHub
tisonkun commented on PR #481: URL: https://github.com/apache/pulsar-site/pull/481#issuecomment-1477655722 We need a full rebuild to regenerate the head metadata for all pages :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] crossoverJie opened a new pull request, #19871: [fix][client] Fix typo in ConsumerStats

2023-03-21 Thread via GitHub
crossoverJie opened a new pull request, #19871: URL: https://github.com/apache/pulsar/pull/19871 ### Motivation This is a minor typo fix. ### Modifications `getTotaBatchReceivedFailed`-> `getTotalBatchReceivedFailed` ### Does this pull request potentially

[GitHub] [pulsar-site] tisonkun opened a new pull request, #477: Replace commerical page with tech docs

2023-03-21 Thread via GitHub
tisonkun opened a new pull request, #477: URL: https://github.com/apache/pulsar-site/pull/477 ### Documentation - [x] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` - [ ] `doc-complete` -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar-site] tisonkun commented on pull request #477: Replace commerical page with tech docs

2023-03-21 Thread via GitHub
tisonkun commented on PR #477: URL: https://github.com/apache/pulsar-site/pull/477#issuecomment-1477362355 I'm going to prepare a new commit to trim all trailing space for docs. So that we don't review such large portion of whitespace changes later. -- This is an automated message from

[GitHub] [pulsar] Anonymitaet commented on pull request #19830: [improve][broker] Authorize originalPrincipal when provided

2023-03-21 Thread via GitHub
Anonymitaet commented on PR #19830: URL: https://github.com/apache/pulsar/pull/19830#issuecomment-1477364840 > I will follow up with docs for these proxy changes. Hi @michaeljmarshall any progress on the doc? Thanks! -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] coderzc opened a new pull request, #19873: [improve][broker][PIP-195] Make bucket merge operation asynchronous

2023-03-21 Thread via GitHub
coderzc opened a new pull request, #19873: URL: https://github.com/apache/pulsar/pull/19873 PIP: #16763 ### Motivation & Modifications Make bucket merge operation asynchronous ### Verifying this change - [x] Make sure that the change passes the CI checks.

[GitHub] [pulsar] github-actions[bot] commented on pull request #19874: [fix][client] Fix NPE when acknowledging multiple messages

2023-03-21 Thread via GitHub
github-actions[bot] commented on PR #19874: URL: https://github.com/apache/pulsar/pull/19874#issuecomment-1477387607 @BewareMyPower Please add the following content to your PR description and select a checkbox: ``` - [ ] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` -

[GitHub] [pulsar] tisonkun commented on issue #19563: [Bug] Metadata ledger creation failed

2023-03-21 Thread via GitHub
tisonkun commented on issue #19563: URL: https://github.com/apache/pulsar/issues/19563#issuecomment-1477443549 Closing... Required information missing. Also, 2.8 is end of support. Please open a new issue if it's still relevant on maintained versions. -- This is an automated

[GitHub] [pulsar] tisonkun closed issue #19563: [Bug] Metadata ledger creation failed

2023-03-21 Thread via GitHub
tisonkun closed issue #19563: [Bug] Metadata ledger creation failed URL: https://github.com/apache/pulsar/issues/19563 -- 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

[GitHub] [pulsar] tisonkun commented on issue #19609: [Bug] Docker build fails if UBUNTU_MIRROR is not set

2023-03-21 Thread via GitHub
tisonkun commented on issue #19609: URL: https://github.com/apache/pulsar/issues/19609#issuecomment-1477442579 Error code 100 means timeout. It's not a bug but network issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] tisonkun closed issue #19609: [Bug] Docker build fails if UBUNTU_MIRROR is not set

2023-03-21 Thread via GitHub
tisonkun closed issue #19609: [Bug] Docker build fails if UBUNTU_MIRROR is not set URL: https://github.com/apache/pulsar/issues/19609 -- 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

[GitHub] [pulsar] yehancha added a comment to the discussion: How to tweak a Pulsar cluster to have a large number of topics with a reasonable throughput

2023-03-21 Thread GitBox
GitHub user yehancha added a comment to the discussion: How to tweak a Pulsar cluster to have a large number of topics with a reasonable throughput The issue seems to be with `DispatchRate` limit. In my case, I need 0.5 b/s throughput on a single topic. I have set the dispatch rate limit to 4

[GitHub] [pulsar] tisonkun commented on issue #6965: Consumer pause do not work

2023-03-21 Thread via GitHub
tisonkun commented on issue #6965: URL: https://github.com/apache/pulsar/issues/6965#issuecomment-1477465847 Closed as answered. -- 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

[GitHub] [pulsar] tisonkun closed issue #6965: Consumer pause do not work

2023-03-21 Thread via GitHub
tisonkun closed issue #6965: Consumer pause do not work URL: https://github.com/apache/pulsar/issues/6965 -- 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,

[GitHub] [pulsar] poorbarcode commented on a diff in pull request #19875: [fix][broker] Fix NPE when update topic policy.

2023-03-21 Thread via GitHub
poorbarcode commented on code in PR #19875: URL: https://github.com/apache/pulsar/pull/19875#discussion_r1143067621 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java: ## @@ -240,7 +240,9 @@ protected void updateTopicPolicy(TopicPolicies data) {

[pulsar-site] branch main updated: [fix][doc] Links and tables (#480)

2023-03-21 Thread tison
This is an automated email from the ASF dual-hosted git repository. tison 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 330fdbc7e69 [fix][doc] Links and tables (#480)

  1   2   3   >