[GitHub] [pulsar] tisonkun closed issue #11282: NPE in OpReadEntry

2023-04-14 Thread via GitHub
tisonkun closed issue #11282: NPE in OpReadEntry URL: https://github.com/apache/pulsar/issues/11282 -- 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 commented on issue #11282: NPE in OpReadEntry

2023-04-14 Thread via GitHub
tisonkun commented on issue #11282: URL: https://github.com/apache/pulsar/issues/11282#issuecomment-1508182109 Closed by https://github.com/apache/pulsar/pull/15837. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar] tisonkun closed issue #10724: [question],An error occurred while testing function mesh

2023-04-14 Thread via GitHub
tisonkun closed issue #10724: [question],An error occurred while testing function mesh URL: https://github.com/apache/pulsar/issues/10724 -- 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] tisonkun commented on issue #10724: [question],An error occurred while testing function mesh

2023-04-14 Thread via GitHub
tisonkun commented on issue #10724: URL: https://github.com/apache/pulsar/issues/10724#issuecomment-1508161432 Closed as this is not the correct repo to report function-mesh issues. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] tisonkun commented on issue #10991: unable to use mTLS to v2.8.0 proxy

2023-04-14 Thread via GitHub
tisonkun commented on issue #10991: URL: https://github.com/apache/pulsar/issues/10991#issuecomment-1508160022 Closing as stale. If it's still relevant to maintained versions, feel free to open a new issue -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] tisonkun closed issue #10991: unable to use mTLS to v2.8.0 proxy

2023-04-14 Thread via GitHub
tisonkun closed issue #10991: unable to use mTLS to v2.8.0 proxy URL: https://github.com/apache/pulsar/issues/10991 -- 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 a diff in pull request #20100: [fix][tool] ConnectorDocGenerator support Java 9+

2023-04-14 Thread via GitHub
tisonkun commented on code in PR #20100: URL: https://github.com/apache/pulsar/pull/20100#discussion_r1166504628 ## pulsar-io/elastic-search/src/main/java/org/apache/pulsar/io/elasticsearch/ElasticSearchConfig.java: ## @@ -242,6 +242,11 @@ public class ElasticSearchConfig

[GitHub] [pulsar] tisonkun opened a new pull request, #20100: [fix][tool] ConnectorDocGenerator support Java 9+

2023-04-14 Thread via GitHub
tisonkun opened a new pull request, #20100: URL: https://github.com/apache/pulsar/pull/20100 Java 9 changes ThreadContextClassLoader and SystemClassLoader to AppClassLoader from URLClassLoader. We should use `java.class.path` instead to get the related URLs. ### Does this pull

[GitHub] [pulsar-site] RobertIndie commented on pull request #515: [Doc] Add C++ doc for client and producer

2023-04-14 Thread via GitHub
RobertIndie commented on PR #515: URL: https://github.com/apache/pulsar-site/pull/515#issuecomment-1508136002 HI @Anonymitaet This change don't need to apply to older doc versions. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] tisonkun commented on issue #18895: [Bug] NoClassDefFoundError BlobStoreBackedReadHandleImpl

2023-04-14 Thread via GitHub
tisonkun commented on issue #18895: URL: https://github.com/apache/pulsar/issues/18895#issuecomment-1508109800 Interesting, `BlobStoreManagedLedgerOffloader` and `BlobStoreBackedReadHandleImpl` are actually both in the NAR file. Now you can load `BlobStoreManagedLedgerOffloader` but

[GitHub] [pulsar] nodece commented on issue #20077: Pulsar client with Authentication using OAuth 2.0 fails to successfully establish connection with the Pulsar instance

2023-04-14 Thread via GitHub
nodece commented on issue #20077: URL: https://github.com/apache/pulsar/issues/20077#issuecomment-1508101780 Could you provide the broker log? -- 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] nodece commented on a diff in pull request #20068: [fix][broker] Fix tenant admin authorization bug.

2023-04-14 Thread via GitHub
nodece commented on code in PR #20068: URL: https://github.com/apache/pulsar/pull/20068#discussion_r1166451517 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java: ## @@ -84,7 +84,18 @@ public void

[GitHub] [pulsar] michaeljmarshall commented on a diff in pull request #20068: [fix][broker] Fix tenant admin authorization bug.

2023-04-14 Thread via GitHub
michaeljmarshall commented on code in PR #20068: URL: https://github.com/apache/pulsar/pull/20068#discussion_r1166326358 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java: ## @@ -84,7 +84,18 @@ public void

[GitHub] [pulsar] nodece commented on a diff in pull request #20068: [fix][broker] Fix tenant admin authorization bug.

2023-04-14 Thread via GitHub
nodece commented on code in PR #20068: URL: https://github.com/apache/pulsar/pull/20068#discussion_r1165799714 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java: ## @@ -84,7 +84,18 @@ public void

[pulsar-client-go] branch master updated: Fix reconnection backoff logic (#1008)

2023-04-14 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz 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 2154d70 Fix reconnection backoff logic

[GitHub] [pulsar] jdfrozen opened a new issue, #20099: [Bug] Delayed message delivery receive duplicate messages when unload topic

2023-04-14 Thread via GitHub
jdfrozen opened a new issue, #20099: URL: https://github.com/apache/pulsar/issues/20099 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Version 2.7.5、2.8.4、master ### Minimal

[GitHub] [pulsar-client-go] BewareMyPower merged pull request #1008: Fix reconnection backoff logic

2023-04-14 Thread via GitHub
BewareMyPower merged PR #1008: URL: https://github.com/apache/pulsar-client-go/pull/1008 -- 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-go] BewareMyPower commented on a diff in pull request #1006: [feat][schema] Support the schema type ProtoNativeSchema

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #1006: URL: https://github.com/apache/pulsar-client-go/pull/1006#discussion_r1166417233 ## pulsar/schema.go: ## @@ -219,6 +226,107 @@ func (ps *ProtoSchema) GetSchemaInfo() *SchemaInfo { return } +type ProtoNativeSchema struct {

[pulsar-site] branch main updated: [fix][doc] RabbitMQ source and sink (#517)

2023-04-14 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 f4b53bf4c50 [fix][doc] RabbitMQ source and sink

[GitHub] [pulsar] tisonkun closed issue #17133: [Doc] Fix the RabbitMQ connector docs

2023-04-14 Thread via GitHub
tisonkun closed issue #17133: [Doc] Fix the RabbitMQ connector docs URL: https://github.com/apache/pulsar/issues/17133 -- 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-site] tisonkun merged pull request #517: [fix][doc] RabbitMQ source and sink

2023-04-14 Thread via GitHub
tisonkun merged PR #517: URL: https://github.com/apache/pulsar-site/pull/517 -- 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] lhotari commented on a diff in pull request #20085: [improve][build] Upgrade snakeyaml version to 2.0

2023-04-14 Thread via GitHub
lhotari commented on code in PR #20085: URL: https://github.com/apache/pulsar/pull/20085#discussion_r1166403989 ## src/owasp-dependency-check-suppressions.xml: ## @@ -39,7 +39,7 @@ e80612549feb5c9191c498de628c1aa80693cf0b CVE-2022-1471

[GitHub] [pulsar-client-node] equanz commented on a diff in pull request #318: [fix][perf] Modify perf_producer to run properly

2023-04-14 Thread via GitHub
equanz commented on code in PR #318: URL: https://github.com/apache/pulsar-client-node/pull/318#discussion_r1166402808 ## perf/perf_producer.js: ## @@ -77,7 +77,7 @@ const Pulsar = require('../index.js'); const numOfMessages = commander.messages; for (let i = 0; i <

[GitHub] [pulsar-client-node] equanz opened a new pull request, #318: [fix][perf] Modify perf_producer to run properly

2023-04-14 Thread via GitHub
equanz opened a new pull request, #318: URL: https://github.com/apache/pulsar-client-node/pull/318 ### Motivation I noticed the `perf_producer` is not working properly as follows. I want to fix it. ``` % node perf/perf_producer.js -u pulsar://localhost:6650 -t

[GitHub] [pulsar-client-go] BewareMyPower commented on a diff in pull request #1006: [feat][schema] Support the schema type ProtoNativeSchema

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #1006: URL: https://github.com/apache/pulsar-client-go/pull/1006#discussion_r1166395173 ## pulsar/schema.go: ## @@ -219,6 +226,107 @@ func (ps *ProtoSchema) GetSchemaInfo() *SchemaInfo { return } +type ProtoNativeSchema struct {

[GitHub] [pulsar] janmee commented on issue #20094: [Bug] Topic ledger id will rollback in some cases

2023-04-14 Thread via GitHub
janmee commented on issue #20094: URL: https://github.com/apache/pulsar/issues/20094#issuecomment-1508044398 it seem in broke.conf,chose zookeeperSessionExpiredPolicy to shutdown can avoid this issue,but it still a problem in the policie reconnect -- This is an automated message from the

[GitHub] [pulsar] tisonkun commented on issue #17133: [Doc] Fix the RabbitMQ connector docs

2023-04-14 Thread via GitHub
tisonkun commented on issue #17133: URL: https://github.com/apache/pulsar/issues/17133#issuecomment-1508043986 I submit https://github.com/apache/pulsar-site/pull/517 to fix the incorrect issue for maintained versions. For missing properties, I'll appreciate it if any further

[GitHub] [pulsar-client-go] BewareMyPower commented on a diff in pull request #1006: [feat][schema] Support the schema type ProtoNativeSchema

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #1006: URL: https://github.com/apache/pulsar-client-go/pull/1006#discussion_r1166390253 ## pulsar/schema.go: ## @@ -219,6 +226,107 @@ func (ps *ProtoSchema) GetSchemaInfo() *SchemaInfo { return } +type ProtoNativeSchema struct {

[GitHub] [pulsar-site] tisonkun opened a new pull request, #517: [fix][doc] RabbitMQ source and sink

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

[pulsar-client-cpp] branch main updated: Fix the static library might failed to link on Windows (#251)

2023-04-14 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 713e254 Fix the static library might

[GitHub] [pulsar-client-cpp] BewareMyPower merged pull request #251: Fix the static library might failed to link on Windows

2023-04-14 Thread via GitHub
BewareMyPower merged PR #251: URL: https://github.com/apache/pulsar-client-cpp/pull/251 -- 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 #19459: [Bug] API /admin/persistent/ return 405 Method Not Allowed

2023-04-14 Thread via GitHub
tisonkun closed issue #19459: [Bug] API /admin/persistent/ return 405 Method Not Allowed URL: https://github.com/apache/pulsar/issues/19459 -- 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] tisonkun commented on issue #19459: [Bug] API /admin/persistent/ return 405 Method Not Allowed

2023-04-14 Thread via GitHub
tisonkun commented on issue #19459: URL: https://github.com/apache/pulsar/issues/19459#issuecomment-1508018432 Closed as answered at https://github.com/streamnative/pulsar/issues/5499#issuecomment-1423802770. Thanks for your investigation @Al-assad! -- This is an automated message

[GitHub] [pulsar] tisonkun commented on issue #20077: Pulsar client with Authentication using OAuth 2.0 fails to successfully establish connection with the Pulsar instance

2023-04-14 Thread via GitHub
tisonkun commented on issue #20077: URL: https://github.com/apache/pulsar/issues/20077#issuecomment-1508014131 cc @nodece does this issue seem to be an auth config issue? Or network issues since I found: >

[GitHub] [pulsar] BewareMyPower commented on issue #20022: [Bug] Pulsar Client ignores producer schema

2023-04-14 Thread via GitHub
BewareMyPower commented on issue #20022: URL: https://github.com/apache/pulsar/issues/20022#issuecomment-1508004715 > In the failed message produce, the data -m '{"family" : "genus" }' should have been rejected, The schema validation only happens when the producer connects to the

[GitHub] [pulsar] harikrishna553 commented on issue #20098: [Doc] Redelivery count, Redelivery delay table should be updated correctly

2023-04-14 Thread via GitHub
harikrishna553 commented on issue #20098: URL: https://github.com/apache/pulsar/issues/20098#issuecomment-1507999493 Document is correct -- 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] harikrishna553 closed issue #20098: [Doc] Redelivery count, Redelivery delay table should be updated correctly

2023-04-14 Thread via GitHub
harikrishna553 closed issue #20098: [Doc] Redelivery count, Redelivery delay table should be updated correctly URL: https://github.com/apache/pulsar/issues/20098 -- 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] harikrishna553 opened a new issue, #20098: [Doc] Redelivery count, Redelivery delay table should be updated correctly

2023-04-14 Thread via GitHub
harikrishna553 opened a new issue, #20098: URL: https://github.com/apache/pulsar/issues/20098 ### 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] AnonHxy commented on pull request #20001: [improve][ml] Add Read cache misses metric for ledger

2023-04-14 Thread via GitHub
AnonHxy commented on PR #20001: URL: https://github.com/apache/pulsar/pull/20001#issuecomment-1507987446 > > When there are too many read request to bookie server, we need find out which ledger is reading without cache. > > @AnonHxy - was there not a bookie metric for this? Also, can

[GitHub] [pulsar] michaeljmarshall commented on a diff in pull request #20068: [fix][broker] Fix tenant admin authorization bug.

2023-04-14 Thread via GitHub
michaeljmarshall commented on code in PR #20068: URL: https://github.com/apache/pulsar/pull/20068#discussion_r1166326358 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java: ## @@ -84,7 +84,18 @@ public void

[GitHub] [pulsar] michaeljmarshall commented on a diff in pull request #20068: [fix][broker] Fix tenant admin authorization bug.

2023-04-14 Thread via GitHub
michaeljmarshall commented on code in PR #20068: URL: https://github.com/apache/pulsar/pull/20068#discussion_r1166323853 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java: ## @@ -84,7 +84,18 @@ public void

<    1   2