[GitHub] [pulsar] mattisonchao merged pull request #20075: [fix][broker] Fix avoid future of clear delayed message can't complete

2023-04-14 Thread via GitHub
mattisonchao merged PR #20075: URL: https://github.com/apache/pulsar/pull/20075 -- 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] branch master updated: [fix][broker] Fix avoid future of clear delayed message can't complete (#20075)

2023-04-14 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao 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 6152cc82ac8 [fix][broker] Fix avoid

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

2023-04-14 Thread via GitHub
yaalsn commented on issue #19609: URL: https://github.com/apache/pulsar/issues/19609#issuecomment-1509515603 @tisonkun I think we should use the official mirrors. -- 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] github-actions[bot] commented on pull request #20110: [improve][broker] Fix ServiceUnitStateCompactionStrategy to cover fast-forward cursor behavior after compaction

2023-04-14 Thread via GitHub
github-actions[bot] commented on PR #20110: URL: https://github.com/apache/pulsar/pull/20110#issuecomment-1509493747 @heesung-sn Please add the following content to your PR description and select a checkbox: ``` - [ ] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` - [ ]

[GitHub] [pulsar] heesung-sn opened a new pull request, #20110: [improve][broker] Fix ServiceUnitStateCompactionStrategy to cover fast-forward cursor behavior after compaction

2023-04-14 Thread via GitHub
heesung-sn opened a new pull request, #20110: URL: https://github.com/apache/pulsar/pull/20110 Fixes #xyz Master Issue: #xyz PIP: #xyz ### Motivation ### Modifications ### Verifying this

[GitHub] [pulsar] github-actions[bot] commented on pull request #19662: [feat][txn] Support for idempotent commit and abort, Solution1.

2023-04-14 Thread via GitHub
github-actions[bot] commented on PR #19662: URL: https://github.com/apache/pulsar/pull/19662#issuecomment-1509459381 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] github-actions[bot] commented on issue #19744: PIP-255: Support for Idempotent Commit and Abort.

2023-04-14 Thread via GitHub
github-actions[bot] commented on issue #19744: URL: https://github.com/apache/pulsar/issues/19744#issuecomment-1509459301 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on issue #19792: Flaky-test: org.apache.pulsar.broker.stats.MetadataStoreStatsTest.testBatchMetadataStoreMetrics

2023-04-14 Thread via GitHub
github-actions[bot] commented on issue #19792: URL: https://github.com/apache/pulsar/issues/19792#issuecomment-1509459260 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on pull request #19794: [fix][authentication][branch-2.10] Store the original authentication data

2023-04-14 Thread via GitHub
github-actions[bot] commented on PR #19794: URL: https://github.com/apache/pulsar/pull/19794#issuecomment-1509459238 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] github-actions[bot] commented on issue #19814: [Bug] [txn] Consumer can receive txn aborted message when readType is replay

2023-04-14 Thread via GitHub
github-actions[bot] commented on issue #19814: URL: https://github.com/apache/pulsar/issues/19814#issuecomment-1509459198 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-site] momo-jun commented on pull request #513: matrix page

2023-04-14 Thread via GitHub
momo-jun commented on PR #513: URL: https://github.com/apache/pulsar-site/pull/513#issuecomment-1509422602 Thanks Lili. Add a few questions: 1. How to maintain the status in these tables? We need a user-friendly way so that maintainers can update the table on a regular basis. 2. Can

[GitHub] [pulsar] michaeljmarshall commented on a diff in pull request #19461: [improve][authorization] Uniformly use allowTopicOperationAsync to check permissions

2023-04-14 Thread via GitHub
michaeljmarshall commented on code in PR #19461: URL: https://github.com/apache/pulsar/pull/19461#discussion_r1167295009 ## pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java: ## @@ -129,11 +142,15 @@ CompletableFuture

[GitHub] [pulsar] onobc commented on a diff in pull request #20106: [fix][admin] Add javax.xml to shade

2023-04-14 Thread via GitHub
onobc commented on code in PR #20106: URL: https://github.com/apache/pulsar/pull/20106#discussion_r1167274462 ## pulsar-client-admin-shaded/pom.xml: ## @@ -131,6 +131,7 @@ com.yahoo.datasketches:sketches-core org.glassfish.jersey*:*

[GitHub] [pulsar-adapters] swamymavuri commented on pull request #46: [Issue #45] [pulsar-client-kafka-compat] Handled Kafka record headers…

2023-04-14 Thread via GitHub
swamymavuri commented on PR #46: URL: https://github.com/apache/pulsar-adapters/pull/46#issuecomment-1509121809 @cbornet , Can you please take a look at this PR and let me know if i need to add any -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] hpvd added a comment to the discussion: Pulsar Shell: native image with GraalVM

2023-04-14 Thread GitBox
GitHub user hpvd added a comment to the discussion: Pulsar Shell: native image with GraalVM could Quarkus provide some help moving in this direction? see > **Quarkus tailors your application for GraalVM** and HotSpot. Amazingly fast > boot time, incredibly low RSS memory (not just heap

[GitHub] [pulsar] aymkhalil commented on a diff in pull request #20104: [fix][schema] Apply Schema for GenericJsonRecord if present

2023-04-14 Thread via GitHub
aymkhalil commented on code in PR #20104: URL: https://github.com/apache/pulsar/pull/20104#discussion_r1167159803 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/generic/GenericJsonRecord.java: ## @@ -39,17 +42,20 @@ public class GenericJsonRecord extends

[GitHub] [pulsar] michaeljmarshall opened a new issue, #20108: Provide way to invalidate an AuthenticationProvider's Cache

2023-04-14 Thread via GitHub
michaeljmarshall opened a new issue, #20108: URL: https://github.com/apache/pulsar/issues/20108 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Motivation When an unexpected signing

[GitHub] [pulsar-client-cpp] BewareMyPower commented on pull request #230: [feat] Support consumer batch receive on C client.

2023-04-14 Thread via GitHub
BewareMyPower commented on PR #230: URL: https://github.com/apache/pulsar-client-cpp/pull/230#issuecomment-1509049662 After thinking for a while, I think the APIs should also be considered carefully. And we should fix the memory leak. So I opened another PR

[GitHub] [pulsar] michaeljmarshall opened a new issue, #20107: Support invalidating client's OAuth2 token after receiving an authentication exception

2023-04-14 Thread via GitHub
michaeljmarshall opened a new issue, #20107: URL: https://github.com/apache/pulsar/issues/20107 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Motivation The `AuthenticationOAuth2`

[GitHub] [pulsar-client-cpp] BewareMyPower opened a new pull request, #252: [C API] Support synchronous consumer batch receive

2023-04-14 Thread via GitHub
BewareMyPower opened a new pull request, #252: URL: https://github.com/apache/pulsar-client-cpp/pull/252 ### Motivation This is the 1st PR to support the batch receive feature for C APIs. ### Modifications - Add the `pulsar_messages_t` struct to represent a list of

[GitHub] [pulsar] aymkhalil commented on issue #20092: [Bug] Pulsar client AUTO_CONSUME "double" as "BigDecimal" in JSON schemas

2023-04-14 Thread via GitHub
aymkhalil commented on issue #20092: URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1509027797 Thanks @tisonkun - I verified your patch locally and it works well. I think it is reasonable because IIUC the `USE_BIG_DECIMAL_FOR_FLOATS` works for deserialization and

[GitHub] [pulsar] nodece commented on issue #19856: Update jaxb and jax-rs libs

2023-04-14 Thread via GitHub
nodece commented on issue #19856: URL: https://github.com/apache/pulsar/issues/19856#issuecomment-1509008438 Hi @onobc, I made https://github.com/apache/pulsar/pull/20106 to fix this issue, could you confirm this PR? -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] nodece opened a new pull request, #20106: [fix][admin] Add javax.xml to shade

2023-04-14 Thread via GitHub
nodece opened a new pull request, #20106: URL: https://github.com/apache/pulsar/pull/20106 Fixes #19856 ### Motivation `javax.xml.bind` didn't package into the shade package. ### Modifications Add `javax.xml.bind` to `pulsar-client-admin-shaded/pom.xml`.

[GitHub] [pulsar] onobc commented on issue #19856: Update jaxb and jax-rs libs

2023-04-14 Thread via GitHub
onobc commented on issue #19856: URL: https://github.com/apache/pulsar/issues/19856#issuecomment-1508997866 Thanks for the info @tisonkun , If it comes down to just finding a resource to implement I could probably volunteer for that as this is quite important to our usage of the

[GitHub] [pulsar] tisonkun commented on issue #19856: Update jaxb and jax-rs libs

2023-04-14 Thread via GitHub
tisonkun commented on issue #19856: URL: https://github.com/apache/pulsar/issues/19856#issuecomment-1508981321 Maybe I can cc the release manager @RobertIndie @poorbarcode @cbornet here for advice. This can impact how users integrate with 3.0.0 but I understand that we don't have too much

[GitHub] [pulsar] tisonkun commented on issue #19856: Update jaxb and jax-rs libs

2023-04-14 Thread via GitHub
tisonkun commented on issue #19856: URL: https://github.com/apache/pulsar/issues/19856#issuecomment-1508980136 For 3.0.0, the schedule is set as: * 2023-04-11 - RC-1 * 2023-04-18 - RC-2 * 2023-04-25 - RC-3 * 2023-05-02 - Announce 3.0 Release You can subscribe to

[GitHub] [pulsar] onobc commented on issue #19856: Update jaxb and jax-rs libs

2023-04-14 Thread via GitHub
onobc commented on issue #19856: URL: https://github.com/apache/pulsar/issues/19856#issuecomment-1508975272 I just noticed that the `3.0.0` milestone only has 2 issues associated w/ it. Can we get this issue in as well? Also, when will there be an RC for `3.0.0`? -- This is an

[GitHub] [pulsar] tisonkun commented on a diff in pull request #20096: [improve][broker] Harden non-null checker

2023-04-14 Thread via GitHub
tisonkun commented on code in PR #20096: URL: https://github.com/apache/pulsar/pull/20096#discussion_r1167084512 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java: ## @@ -3066,34 +3068,38 @@ boolean shouldCloseLedger(LedgerHandle lh) {

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

2023-04-14 Thread via GitHub
tisonkun commented on issue #19609: URL: https://github.com/apache/pulsar/issues/19609#issuecomment-1508955324 @yaalsn what do you exactly propose here? -- 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-client-cpp] BewareMyPower commented on a diff in pull request #230: [feat] Support consumer batch receive on C client.

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #230: URL: https://github.com/apache/pulsar-client-cpp/pull/230#discussion_r1167064674 ## tests/c/c_ConsumerTest.cc: ## @@ -0,0 +1,78 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #230: [feat] Support consumer batch receive on C client.

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #230: URL: https://github.com/apache/pulsar-client-cpp/pull/230#discussion_r1167053780 ## tests/c/c_ConsumerTest.cc: ## @@ -0,0 +1,78 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license

[GitHub] [pulsar-client-cpp] shibd commented on a diff in pull request #230: [feat] Support consumer batch receive on C client.

2023-04-14 Thread via GitHub
shibd commented on code in PR #230: URL: https://github.com/apache/pulsar-client-cpp/pull/230#discussion_r1166984212 ## tests/c/c_ConsumerTest.cc: ## @@ -0,0 +1,78 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements.

[GitHub] [pulsar] niharrathod commented on issue #19456: Flaky-test: MetadataStoreStatsTest.testMetadataStoreStats

2023-04-14 Thread via GitHub
niharrathod commented on issue #19456: URL: https://github.com/apache/pulsar/issues/19456#issuecomment-1508767436 @tisonkun : I agree, I was just sharing info :). I just looked at it again and I can see, there is fix [PR#19829](https://github.com/apache/pulsar/pull/19829) around

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-14 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1166912389 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -567,26 +574,19 @@ public synchronized

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-14 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1166912389 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -567,26 +574,19 @@ public synchronized

[pulsar-client-cpp] branch main updated: [feat] Support set consumer regex subscription mode for C client. (#247)

2023-04-14 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi 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 4d659a8 [feat] Support set consumer

[GitHub] [pulsar-client-cpp] shibd merged pull request #247: [feat] Support set consumer regex subscription mode for C client.

2023-04-14 Thread via GitHub
shibd merged PR #247: URL: https://github.com/apache/pulsar-client-cpp/pull/247 -- 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] frankjkelly opened a new issue, #1010: Seeing high pulsar_client_consumer_processing_time_seconds_bucket values

2023-04-14 Thread via GitHub
frankjkelly opened a new issue, #1010: URL: https://github.com/apache/pulsar-client-go/issues/1010 Expected behavior Query we are running is ``` histogram_quantile(0.99, sum(rate(pulsar_client_consumer_processing_time_seconds_bucket{x,pulsar_namespace=~'yy',

[GitHub] [pulsar] tiny-rain commented on issue #20105: [Bug] ZKMetadataStore runs into deadlock

2023-04-14 Thread via GitHub
tiny-rain commented on issue #20105: URL: https://github.com/apache/pulsar/issues/20105#issuecomment-1508391757 `Java stack information for the threads listed above: === "metadata-store-40-1": at

[GitHub] [pulsar] tiny-rain opened a new issue, #20105: [Bug] ZKMetadataStore runs into deadlock

2023-04-14 Thread via GitHub
tiny-rain opened a new issue, #20105: URL: https://github.com/apache/pulsar/issues/20105 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Version Pulsar version: 2.11 OS: MacOs

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #20096: [improve][broker] Harden non-null checker

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #20096: URL: https://github.com/apache/pulsar/pull/20096#discussion_r1166737860 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java: ## @@ -3066,34 +3068,38 @@ boolean shouldCloseLedger(LedgerHandle lh)

[GitHub] [pulsar-client-cpp] BewareMyPower commented on pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#issuecomment-1508382123 @shibd Thanks for your suggestions. I'm working on simplifying all test scripts into the `run-unit-tests.sh` currently. It would also support configuring

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r1166706278 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar] codecov-commenter commented on pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-14 Thread via GitHub
codecov-commenter commented on PR #20086: URL: https://github.com/apache/pulsar/pull/20086#issuecomment-1508339284 ## [Codecov](https://codecov.io/gh/apache/pulsar/pull/20086?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)

[GitHub] [pulsar] tisonkun commented on issue #20092: [Bug] Pulsar client AUTO_CONSUME "double" as "BigDecimal" in JSON schemas

2023-04-14 Thread via GitHub
tisonkun commented on issue #20092: URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1508322846 I create a patch to fix your use case https://github.com/apache/pulsar/pull/20104. You can give it a try and help in review. I don't know if it affects other use case but

[GitHub] [pulsar] tisonkun opened a new pull request, #20104: [fix][schema] Apply Schema for GenericJsonRecord if present

2023-04-14 Thread via GitHub
tisonkun opened a new pull request, #20104: URL: https://github.com/apache/pulsar/pull/20104 This closes https://github.com/apache/pulsar/issues/20092. ### Motivation See https://github.com/apache/pulsar/issues/20092 ### Modifications Add `jackson-dataformat-avro`

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r1166679474 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] shibd commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
shibd commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r117184 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r1166655369 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r111330 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] BewareMyPower commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r1166655369 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] shibd commented on pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
shibd commented on PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#issuecomment-1508283008 > @shibd You're right. I think we need to support this format. It seems simple, can you include it in this PR? -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar-client-cpp] shibd commented on a diff in pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
shibd commented on code in PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#discussion_r1166643609 ## .github/workflows/ci-pr-validation.yaml: ## @@ -93,6 +93,14 @@ jobs: - name: Build run: make -j8 + - name: Run OAuth2 tests +

[GitHub] [pulsar-client-cpp] BewareMyPower commented on pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
BewareMyPower commented on PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#issuecomment-1508280781 @shibd You're right. I think we need to support this format. -- 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 #20092: [Bug] Pulsar client AUTO_CONSUME "double" as "BigDecimal" in JSON schemas

2023-04-14 Thread via GitHub
tisonkun commented on issue #20092: URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1508264901 Basically I don't think it's bad but we should adjust to correct schema using schema info. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] tisonkun commented on issue #20092: [Bug] Pulsar client AUTO_CONSUME "double" as "BigDecimal" in JSON schemas

2023-04-14 Thread via GitHub
tisonkun commented on issue #20092: URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1508264050 I think it's this PR change the behavior: https://github.com/apache/pulsar/pull/15687 Note that we now use: ```

[GitHub] [pulsar-client-cpp] shibd commented on pull request #249: Support the base64 encoded credentials for OAuth2 authentication

2023-04-14 Thread via GitHub
shibd commented on PR #249: URL: https://github.com/apache/pulsar-client-cpp/pull/249#issuecomment-1508253560 >Support the following URL representations for private_key: - file:///path/to/key/file - data:application/json;base64, Nothing to do with this PR. Do we support

[pulsar-site] branch main updated: Refine the java client doc (#516)

2023-04-14 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu 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 c502b839cb8 Refine the java client doc (#516)

[GitHub] [pulsar-site] Anonymitaet merged pull request #516: Refine the java client doc

2023-04-14 Thread via GitHub
Anonymitaet merged PR #516: URL: https://github.com/apache/pulsar-site/pull/516 -- 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-site] branch main updated: [Doc] Add C++ doc for client and producer (#515)

2023-04-14 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu 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 a7cd17d7152 [Doc] Add C++ doc for client and

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

2023-04-14 Thread via GitHub
Anonymitaet merged PR #515: URL: https://github.com/apache/pulsar-site/pull/515 -- 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 #20092: [Bug] Pulsar client AUTO_CONSUME "double" as "BigDecimal" in JSON schemas

2023-04-14 Thread via GitHub
tisonkun commented on issue #20092: URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1508233139 Thanks for your report @aymkhalil! Can you try out 2.11.0 and build from source? I cannot reproduce this issue with current master 091ee2504ffbe6ec98e354b76e7f4c045e1914aa. --

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

2023-04-14 Thread via GitHub
yaalsn commented on issue #19609: URL: https://github.com/apache/pulsar/issues/19609#issuecomment-1508232352 https://github.com/apache/pulsar/issues/19098 ``` UBUNTU_MIRROR: http://archive.ubuntu.com/ubuntu/ UBUNTU_SECURITY_MIRROR: http://security.ubuntu.com/ubuntu/ ``` --

[GitHub] [pulsar-client-go] tisonkun commented on pull request #910: [improve][ci] change token name to `GITHUB_TOKEN`

2023-04-14 Thread via GitHub
tisonkun commented on PR #910: URL: https://github.com/apache/pulsar-client-go/pull/910#issuecomment-1508226156 ``` rerun-failed-jobs for 'CI'. Follow progress at https://github.com/apache/pulsar-client-go/actions/runs/2812684476 { "message": "Unable to retry this workflow run

[GitHub] [pulsar-client-node] RobertIndie closed issue #254: Install failed on macos x64 using clang

2023-04-14 Thread via GitHub
RobertIndie closed issue #254: Install failed on macos x64 using clang URL: https://github.com/apache/pulsar-client-node/issues/254 -- 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-node] RobertIndie commented on issue #254: Install failed on macos x64 using clang

2023-04-14 Thread via GitHub
RobertIndie commented on issue #254: URL: https://github.com/apache/pulsar-client-node/issues/254#issuecomment-1508224410 We could close this issue as 1.8.0 is already released. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-client-go] tisonkun commented on pull request #823: fix: fix schema error

2023-04-14 Thread via GitHub
tisonkun commented on PR #823: URL: https://github.com/apache/pulsar-client-go/pull/823#issuecomment-1508222497 /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

[GitHub] [pulsar-client-go] tisonkun closed pull request #1009: DNM - test pulsarbot

2023-04-14 Thread via GitHub
tisonkun closed pull request #1009: DNM - test pulsarbot URL: https://github.com/apache/pulsar-client-go/pull/1009 -- 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] tisonkun commented on issue #227: mac m1 install pulsar-client

2023-04-14 Thread via GitHub
tisonkun commented on issue #227: URL: https://github.com/apache/pulsar-client-node/issues/227#issuecomment-1508216973 I can install this library (1.8.1) on Mac M1 now. Closing.. Please open a new issue if there is still something not working. -- This is an automated

[GitHub] [pulsar-client-node] tisonkun closed issue #227: mac m1 install pulsar-client

2023-04-14 Thread via GitHub
tisonkun closed issue #227: mac m1 install pulsar-client URL: https://github.com/apache/pulsar-client-node/issues/227 -- 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-go] tisonkun commented on pull request #1009: DNM - test pulsarbot

2023-04-14 Thread via GitHub
tisonkun commented on PR #1009: URL: https://github.com/apache/pulsar-client-go/pull/1009#issuecomment-1508208279 /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

[pulsar-client-go] branch tisonkun-patch-1 created (now 6ee1b2b)

2023-04-14 Thread tison
This is an automated email from the ASF dual-hosted git repository. tison pushed a change to branch tisonkun-patch-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git at 6ee1b2b Update bot.yml No new revisions were added by this update.

[pulsar-client-go] branch master updated: [improve][ci] try to fix bot workflow (#910)

2023-04-14 Thread tison
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-client-go.git The following commit(s) were added to refs/heads/master by this push: new 55f0c7c [improve][ci] try to fix bot

[GitHub] [pulsar-client-go] tisonkun merged pull request #910: [improve][ci] change token name to `GITHUB_TOKEN`

2023-04-14 Thread via GitHub
tisonkun merged PR #910: URL: https://github.com/apache/pulsar-client-go/pull/910 -- 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] tisonkun closed issue #909: [CI] pulsar bot not work in this repo due to `GO_CLIENT_BOT_TOKEN` missing

2023-04-14 Thread via GitHub
tisonkun closed issue #909: [CI] pulsar bot not work in this repo due to `GO_CLIENT_BOT_TOKEN` missing URL: https://github.com/apache/pulsar-client-go/issues/909 -- 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-client-go] tisonkun commented on pull request #910: [improve][ci] change token name to `GITHUB_TOKEN`

2023-04-14 Thread via GitHub
tisonkun commented on PR #910: URL: https://github.com/apache/pulsar-client-go/pull/910#issuecomment-1508207104 This change doesn't make the case worse. Let's give it a try. If it still won't work, we can find other approach or just remove the workflow since we actually live well

[GitHub] [pulsar-client-go] tisonkun closed pull request #1009: DNM - test pulsarbot

2023-04-14 Thread via GitHub
tisonkun closed pull request #1009: DNM - test pulsarbot URL: https://github.com/apache/pulsar-client-go/pull/1009 -- 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-go] tisonkun commented on pull request #1009: DNM - test pulsarbot

2023-04-14 Thread via GitHub
tisonkun commented on PR #1009: URL: https://github.com/apache/pulsar-client-go/pull/1009#issuecomment-1508204142 /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

[pulsar-client-go] branch tisonkun-patch-1 updated (95b99eb -> 6ee1b2b)

2023-04-14 Thread tison
This is an automated email from the ASF dual-hosted git repository. tison pushed a change to branch tisonkun-patch-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git from 95b99eb DNM - test pulsarbot add 6ee1b2b Update bot.yml No new revisions were added by

[pulsar-client-go] branch tisonkun-patch-1 created (now 95b99eb)

2023-04-14 Thread tison
This is an automated email from the ASF dual-hosted git repository. tison pushed a change to branch tisonkun-patch-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git at 95b99eb DNM - test pulsarbot This branch includes the following new commits: new 95b99eb

[GitHub] [pulsar-client-go] tisonkun commented on pull request #1009: DNM - test pulsarbot

2023-04-14 Thread via GitHub
tisonkun commented on PR #1009: URL: https://github.com/apache/pulsar-client-go/pull/1009#issuecomment-1508200574 /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

[pulsar-client-go] 01/01: DNM - test pulsarbot

2023-04-14 Thread tison
This is an automated email from the ASF dual-hosted git repository. tison pushed a commit to branch tisonkun-patch-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git commit 95b99ebe9d9b39f03677d62ffd1185398efc687d Author: tison AuthorDate: Fri Apr 14 17:11:53 2023 +0800

[GitHub] [pulsar] codelipenghui added a comment to the discussion: [question]How many consumers or producers can be created on a pulsar client?

2023-04-14 Thread GitBox
GitHub user codelipenghui added a comment to the discussion: [question]How many consumers or producers can be created on a pulsar client? The issue had no activity for 30 days, mark with Stale label. GitHub link: https://github.com/apache/pulsar/discussions/20103#discussioncomment-5613741

[GitHub] [pulsar] wmccarley added a comment to the discussion: [question]How many consumers or producers can be created on a pulsar client?

2023-04-14 Thread GitBox
GitHub user wmccarley added a comment to the discussion: [question]How many consumers or producers can be created on a pulsar client? In the Java implementation the `PulsarClient` class stores producer and consumer instances in a set backed by a `ConcurrentHashMap` so the count is

[GitHub] [pulsar] tisonkun closed issue #11227: [question]How many consumers or producers can be created on a pulsar client?

2023-04-14 Thread via GitHub
tisonkun closed issue #11227: [question]How many consumers or producers can be created on a pulsar client? URL: https://github.com/apache/pulsar/issues/11227 -- 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] [pulsar] xuesongxs created a discussion: [question]How many consumers or producers can be created on a pulsar client?

2023-04-14 Thread GitBox
GitHub user xuesongxs created a discussion: [question]How many consumers or producers can be created on a pulsar client? How many consumers or producers can be created on a pulsar client? GitHub link: https://github.com/apache/pulsar/discussions/20103 This is an automatically sent email

[GitHub] [pulsar] tisonkun closed issue #10623: function down suddenly

2023-04-14 Thread via GitHub
tisonkun closed issue #10623: function down suddenly URL: https://github.com/apache/pulsar/issues/10623 -- 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 #10623: function down suddenly

2023-04-14 Thread via GitHub
tisonkun commented on issue #10623: URL: https://github.com/apache/pulsar/issues/10623#issuecomment-1508189921 Vague to investigate. Please open a new issue with more details about the repro and symptom if it's still relevant to maintained versions. -- This is an automated message from

[GitHub] [pulsar] codelipenghui added a comment to the discussion: Develop for Java Client Plusar Statistics

2023-04-14 Thread GitBox
GitHub user codelipenghui added a comment to the discussion: Develop for Java Client Plusar Statistics The issue had no activity for 30 days, mark with Stale label. GitHub link: https://github.com/apache/pulsar/discussions/20102#discussioncomment-5613725 This is an automatically sent

[GitHub] [pulsar] tisonkun closed issue #10519: Develop for Java Client Plusar Statistics

2023-04-14 Thread via GitHub
tisonkun closed issue #10519: Develop for Java Client Plusar Statistics URL: https://github.com/apache/pulsar/issues/10519 -- 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] complone created a discussion: Develop for Java Client Plusar Statistics

2023-04-14 Thread GitBox
GitHub user complone created a discussion: Develop for Java Client Plusar Statistics **Is your feature request related to a problem? Please describe.** A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] **Describe the solution you'd like** I am

[GitHub] [pulsar] tisonkun closed issue #18158: [Bug] expireTimeOfIncompleteChunkedMessage(1, TimeUnit.MILLISECONDS) has bug

2023-04-14 Thread via GitHub
tisonkun closed issue #18158: [Bug] expireTimeOfIncompleteChunkedMessage(1, TimeUnit.MILLISECONDS) has bug URL: https://github.com/apache/pulsar/issues/18158 -- 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] [pulsar] tisonkun commented on issue #18158: [Bug] expireTimeOfIncompleteChunkedMessage(1, TimeUnit.MILLISECONDS) has bug

2023-04-14 Thread via GitHub
tisonkun commented on issue #18158: URL: https://github.com/apache/pulsar/issues/18158#issuecomment-1508186671 Do not how to reproduce. Please open a new issue with well-formatted log and reasoning. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #20085: [improve][build] Upgrade snakeyaml version to 2.0

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

[GitHub] [pulsar] prime-minister-of-fun added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2023-04-14 Thread GitBox
GitHub user prime-minister-of-fun added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x Here's the output from running pulsar standalone. I stopped it towards the end: `2022-02-15T20:07:57,250+ [main] INFO

[GitHub] [pulsar] github-actions[bot] added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2023-04-14 Thread GitBox
GitHub user github-actions[bot] added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x The issue had no activity for 30 days, mark with Stale label. GitHub link: https://github.com/apache/pulsar/discussions/20101#discussioncomment-5613700 This is an

[GitHub] [pulsar] github-actions[bot] added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2023-04-14 Thread GitBox
GitHub user github-actions[bot] added a comment to the discussion: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x The issue had no activity for 30 days, mark with Stale label. GitHub link: https://github.com/apache/pulsar/discussions/20101#discussioncomment-5613701 This is an

[GitHub] [pulsar] tisonkun closed issue #14305: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2023-04-14 Thread via GitHub
tisonkun closed issue #14305: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x URL: https://github.com/apache/pulsar/issues/14305 -- 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 pull request #12396: Fix NPE when create OpEntry

2023-04-14 Thread via GitHub
tisonkun commented on PR #12396: URL: https://github.com/apache/pulsar/pull/12396#issuecomment-1508182416 Fixed 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 URL

svn commit: r61259 - /dev/pulsar/pulsar-client-node/pulsar-client-node-1.8.2-rc.4/

2023-04-14 Thread baodi
Author: baodi Date: Fri Apr 14 09:00:59 2023 New Revision: 61259 Log: Staging artifacts and signature for Pulsar Node.js client release 1.8.2-rc.4 Added: dev/pulsar/pulsar-client-node/pulsar-client-node-1.8.2-rc.4/

  1   2   >