michaeljmarshall commented on pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#issuecomment-801675884
@aahmed-se - can you explain why clearing the metric resolves the flakiness?
Is it already initialized because of concurrent tests or tests that have
already run in the
zymap commented on pull request #9834:
URL: https://github.com/apache/pulsar/pull/9834#issuecomment-801673074
@Jennifer88huang @Anonymitaet PTAL. Thanks
This is an automated message from the Apache Git Service.
To respond to
xuesongxs closed issue #9952:
URL: https://github.com/apache/pulsar/issues/9952
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
xuesongxs opened a new issue #9953:
URL: https://github.com/apache/pulsar/issues/9953
**Describe the bug**
topicName: persistent://public/default/test-string5
Whe
yitian108 edited a comment on issue #9796:
URL: https://github.com/apache/pulsar/issues/9796#issuecomment-801670462
[pulsarTest.tar.gz](https://github.com/apache/pulsar/files/6161780/pulsarTest.tar.gz)
hi, I upload the test project which built with gcc 4.8.5, BTW, I update the
versi
yitian108 commented on issue #9796:
URL: https://github.com/apache/pulsar/issues/9796#issuecomment-801670462
[pulsarTest.tar.gz](https://github.com/apache/pulsar/files/6161780/pulsarTest.tar.gz)
hi, I upload the test project which built with gcc 4.8.5
--
xuesongxs opened a new issue #9952:
URL: https://github.com/apache/pulsar/issues/9952
**Describe the bug**
topicName: persistent://public/default/test-string5
Whe
BewareMyPower commented on issue #9922:
URL: https://github.com/apache/pulsar/issues/9922#issuecomment-801669922
Currently `AVRO`, `JSON` and `PROTOBUF` (not `PROTOBUF_NATIVE`) share the
same schema validator. See
https://github.com/apache/pulsar/blob/7d3317e290ca34918e10379b6592e28678cde9
sijie commented on pull request #9895:
URL: https://github.com/apache/pulsar/pull/9895#issuecomment-801664166
The concerns will still be around AUTO_SCHEMA vs OBJECT. It will cause
confusion. IMO, we should just have one schema type that is used to
automatically get the "object" based on t
Huanli-Meng opened a new pull request #9951:
URL: https://github.com/apache/pulsar/pull/9951
### Motivation
According to this [PR](https://github.com/apache/pulsar/pull/9678), the DLQ
is supported in the key_shared sub. Therefore, update the docs accordingly.
### Modificat
eolivelli commented on pull request #9895:
URL: https://github.com/apache/pulsar/pull/9895#issuecomment-801656118
@sijie did you see the latest version of this patch here?
I have refactored the initial patch, now there is no code duplication and we
have AbstractAutoConsumeSchema inst
jiazhai commented on pull request #9924:
URL: https://github.com/apache/pulsar/pull/9924#issuecomment-801642189
/pulsarbot run-failure-checks
This is an automated message from the Apache Git Service.
To respond to the message
softwareace opened a new issue #9950:
URL: https://github.com/apache/pulsar/issues/9950
Direct quote from the code in crc32c_sw
/* Table-driven software version as a fall-back. This is about 15 times
slower
than using the hardware instructions. This assumes little-endian integers,
zymap commented on a change in pull request #9834:
URL: https://github.com/apache/pulsar/pull/9834#discussion_r596563144
##
File path: site2/website/release-notes.md
##
@@ -1,6 +1,181 @@
## Apache Pulsar Release Notes
+### 2.7.1 — 2020-11-25
Review comment:
```sug
Anonymitaet commented on a change in pull request #9834:
URL: https://github.com/apache/pulsar/pull/9834#discussion_r596553790
##
File path: site2/website/release-notes.md
##
@@ -1,6 +1,181 @@
## Apache Pulsar Release Notes
+### 2.7.1 — 2020-11-25
+
+ Broker
+
+- Fix
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-801621751
Had merge conflicts. Fixed with a rebase.
This is an automated message from the Apache Git Service.
To r
michaeljmarshall commented on issue #3804:
URL: https://github.com/apache/pulsar/issues/3804#issuecomment-801619369
@MarvinCai, @sijie, @eolivelli - It looks like the feature to improve read
throughput for blob storage is still outstanding. I'm very interested in
helping to contribute this
codelipenghui commented on pull request #9886:
URL: https://github.com/apache/pulsar/pull/9886#issuecomment-801610142
ok
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
codelipenghui commented on issue #9922:
URL: https://github.com/apache/pulsar/issues/9922#issuecomment-801609830
@BewareMyPower Could you please help take a look at this issue?
This is an automated message from the Apache Git
devinbost commented on issue #6054:
URL: https://github.com/apache/pulsar/issues/6054#issuecomment-801609488
After taking a closer look at #7266, it's not clear if it will resolve the
issue because we're seeing this issue with functions running with exclusive
subscriptions, not shared subs
dlg99 commented on a change in pull request #9947:
URL: https://github.com/apache/pulsar/pull/9947#discussion_r596539615
##
File path:
pulsar-functions/api-java/src/main/java/org/apache/pulsar/functions/api/Record.java
##
@@ -77,6 +77,15 @@
return Optional.empty();
sijie commented on pull request #9895:
URL: https://github.com/apache/pulsar/pull/9895#issuecomment-801606979
`getField` will return an empty list. `schemaType` gives the schema type.
`GenericRecord` is just a container to hold either a primitive value or a
struct. My main point here is to
sijie commented on a change in pull request #9947:
URL: https://github.com/apache/pulsar/pull/9947#discussion_r596538029
##
File path:
pulsar-functions/api-java/src/main/java/org/apache/pulsar/functions/api/Record.java
##
@@ -77,6 +77,15 @@
return Optional.empty();
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 3bb9c54 [Doc] Add guides for TLS SNI support (cpp and python clients)
(#9937)
add 7d3317e [CI] Fix Transaction
freeznet commented on pull request #9945:
URL: https://github.com/apache/pulsar/pull/9945#issuecomment-801605722
@jerrypeng Thanks for bring this here, and I get your point. I would also
like to express my opinion here.
The added knob is more concern from user's perspective, like if
This is an automated email from the ASF dual-hosted git repository.
aahmed pushed a change to branch prometheusMtest
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
discard d822255 Cleanup up PromethuesMetricsTest
add 8e34089 Cleanup up PromethuesMetricsTest
This update ad
aahmed-se commented on a change in pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#discussion_r596537933
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java
##
@@ -99,7 +100,8 @@
@
sijie merged pull request #9934:
URL: https://github.com/apache/pulsar/pull/9934
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
jiazhai commented on pull request #9924:
URL: https://github.com/apache/pulsar/pull/9924#issuecomment-801605324
rebase the branch with the latest master would help on the CI
This is an automated message from the Apache Git Se
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from f49d094 [Doc] Add guides for producer access mode (#9938)
add 3bb9c54 [Doc] Add guides for TLS SNI support (cpp
sijie merged pull request #9937:
URL: https://github.com/apache/pulsar/pull/9937
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
BewareMyPower commented on issue #9796:
URL: https://github.com/apache/pulsar/issues/9796#issuecomment-801603008
Can you upload your library (built with GCC 4.8.5)? Then I could try it in
my CentOS 7 environment. BTW, I assume your library links dependencies
dynamically.
> Attach fi
aahmed-se commented on a change in pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#discussion_r596536194
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java
##
@@ -99,7 +100,8 @@
@
sakurafly123 commented on a change in pull request #9910:
URL: https://github.com/apache/pulsar/pull/9910#discussion_r596533833
##
File path:
pulsar-sql/presto-pulsar/src/main/java/org/apache/pulsar/sql/presto/PulsarConnectorCache.java
##
@@ -112,7 +112,7 @@ private static Man
sakurafly123 commented on a change in pull request #9910:
URL: https://github.com/apache/pulsar/pull/9910#discussion_r596531770
##
File path:
pulsar-sql/presto-pulsar/src/main/java/org/apache/pulsar/sql/presto/PulsarConnectorCache.java
##
@@ -112,7 +112,7 @@ private static Man
Anonymitaet commented on pull request #9903:
URL: https://github.com/apache/pulsar/pull/9903#issuecomment-801597264
/pulsarbot run-failure-checks
This is an automated message from the Apache Git Service.
To respond to the mes
merlimat commented on a change in pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#discussion_r596526863
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java
##
@@ -99,7 +100,8 @@
@O
yitian108 commented on issue #9796:
URL: https://github.com/apache/pulsar/issues/9796#issuecomment-801581957
First, I am sorry for the confusion, my bad, actually, I just used the api
setMaxPendingMessages(10)
Second, I list my enviornment for your reference:
- CentOS 7.6
This is an automated email from the ASF dual-hosted git repository.
massakam pushed a change to annotated tag v1.3.0
in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git.
*** WARNING: tag v1.3.0 was modified! ***
from c8f3943 (commit)
to b3dac9e (tag)
tagging c8
This is an automated email from the ASF dual-hosted git repository.
massakam pushed a commit to branch branch-1.3
in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git
The following commit(s) were added to refs/heads/branch-1.3 by this push:
new c8f3943 1.3.0
c8f3943 is
This is an automated email from the ASF dual-hosted git repository.
penghui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new f49d094 [Doc] Add guides for producer access m
codelipenghui merged pull request #9938:
URL: https://github.com/apache/pulsar/pull/9938
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 g
Anonymitaet commented on pull request #9938:
URL: https://github.com/apache/pulsar/pull/9938#issuecomment-801565135
@zymap many thanks for your instructions. I've updated, PTAL.
This is an automated message from the Apache Gi
This is an automated email from the ASF dual-hosted git repository.
yong pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from dedd882 [Documentation] Fix reference configuration table formatting
(#9941)
add fc5768c [Website] Update relea
zymap merged pull request #9949:
URL: https://github.com/apache/pulsar/pull/9949
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
Jennifer88huang commented on a change in pull request #9834:
URL: https://github.com/apache/pulsar/pull/9834#discussion_r596483485
##
File path: site2/website/release-notes.md
##
@@ -1,6 +1,181 @@
## Apache Pulsar Release Notes
+### 2.7.1 — 2020-11-25
+
+ Broker
+
+-
Jennifer88huang commented on pull request #9941:
URL: https://github.com/apache/pulsar/pull/9941#issuecomment-801539240
@michaeljmarshall @eolivelli those configuration info comes from different
conf files. I was wondering whether we could generate the configuration
reference from those co
This is an automated email from the ASF dual-hosted git repository.
hjf pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from c4aaf41 Add TestNG groups to pulsar-broker (#9712)
add dedd882 [Documentation] Fix reference configuration table
Jennifer88huang merged pull request #9941:
URL: https://github.com/apache/pulsar/pull/9941
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
zymap opened a new pull request #9949:
URL: https://github.com/apache/pulsar/pull/9949
---
*Motivation*
Update release.json file to add the 2.7.1 release.
This is an automated message from the Apache Git Ser
zymap commented on pull request #9834:
URL: https://github.com/apache/pulsar/pull/9834#issuecomment-801532773
@Anonymitaet @Huanli-Meng @Jennifer88huang Please take a look when you have
time. Thanks
This is an automated mess
aahmed-se commented on a change in pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#discussion_r596473985
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java
##
@@ -99,7 +100,8 @@
@
merlimat commented on a change in pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948#discussion_r596473130
##
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java
##
@@ -99,7 +100,8 @@
@O
jerrypeng commented on issue #6054:
URL: https://github.com/apache/pulsar/issues/6054#issuecomment-801529366
> Do they batch by default?
Yes, batching at the pulsar client level is enabled when a Pulsar function
is writing messages to the "output" topic.
---
zymap commented on a change in pull request #9938:
URL: https://github.com/apache/pulsar/pull/9938#discussion_r596472637
##
File path: site2/docs/concepts-messaging.md
##
@@ -53,6 +53,25 @@ Producers send messages to brokers synchronously (sync) or
asynchronously (async
| Syn
This is an automated email from the ASF dual-hosted git repository.
aahmed pushed a change to branch prometheusMtest
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
discard c56f50f Cleanup up PromethuesMetricsTest
omit ab8384a Add TestNG groups to pulsar-broker
add c4aa
aahmed-se opened a new pull request #9948:
URL: https://github.com/apache/pulsar/pull/9948
### Motivation
* PrometheusMetricsTest is one of the flaky tests, that needs additional
investigation.
### Modifications
* Clear static metrics to harden tests.
* Remove unnece
This is an automated email from the ASF dual-hosted git repository.
aahmed pushed a change to branch prometheusMtest
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
at c56f50f Cleanup up PromethuesMetricsTest
This branch includes the following new commits:
new c56f50f
This is an automated email from the ASF dual-hosted git repository.
aahmed pushed a commit to branch prometheusMtest
in repository https://gitbox.apache.org/repos/asf/pulsar.git
commit c56f50fbe2dd514ea545882fc19064b5f8fcf2a7
Author: Ali Ahmed
AuthorDate: Wed Mar 17 17:27:10 2021 -0700
Clea
This is an automated email from the ASF dual-hosted git repository.
aahmed pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git.
from 94bf13f [Documentation] Fix ThresholdShedder Typo (#9933)
add c4aaf41 Add TestNG groups to pulsar-broker (#971
aahmed-se merged pull request #9712:
URL: https://github.com/apache/pulsar/pull/9712
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
devinbost commented on issue #6054:
URL: https://github.com/apache/pulsar/issues/6054#issuecomment-801518480
**I found a huge clue on this issue** (in Pulsar 2.6.3).
Several hours after we collected those stats, I got a heap dump on the
function that was consuming from the dead topic, an
jerrypeng commented on pull request #9945:
URL: https://github.com/apache/pulsar/pull/9945#issuecomment-801501295
I do think the tests written as part of
https://github.com/apache/pulsar/pull/9907/files
we should still keep.
---
jerrypeng commented on pull request #9945:
URL: https://github.com/apache/pulsar/pull/9945#issuecomment-801496669
cc @david-streamlio @freeznet
This is an automated message from the Apache Git Service.
To respond to the mess
dlg99 commented on a change in pull request #9927:
URL: https://github.com/apache/pulsar/pull/9927#discussion_r596433829
##
File path:
pulsar-io/kafka-connect-adaptor/src/main/java/org/apache/pulsar/io/kafka/connect/KafkaConnectSink.java
##
@@ -0,0 +1,277 @@
+/**
+ * Licensed
dlg99 opened a new pull request #9947:
URL: https://github.com/apache/pulsar/pull/9947
### Motivation
Looking at https://github.com/apache/pulsar/pull/9927#discussion_r595714308
there is no way to reliably get partition number.
There is Optional getPartitionId() which retu
csthomas1 opened a new issue #9946:
URL: https://github.com/apache/pulsar/issues/9946
**Describe the bug**
I'm able to successfully deploy in Pulsar a Pulsar function that includes a
':' as part of its function name. When I later try to delete the function
deployment, the Pulsar broker
jerrypeng commented on pull request #9945:
URL: https://github.com/apache/pulsar/pull/9945#issuecomment-801473251
The correct solution should just be
https://github.com/apache/pulsar/pull/9943/files
This is an automate
jerrypeng opened a new pull request #9945:
URL: https://github.com/apache/pulsar/pull/9945
…(#9907)"
This reverts commit d0249e5695c75d8d65a4180fd599b17971368f33.
### Motivation
We don't need to add additional configuration for message properties to be
set for Pulsa
jerrypeng commented on pull request #9907:
URL: https://github.com/apache/pulsar/pull/9907#issuecomment-801468702
Why do we need to add a flag for the source? If the `getProperties()` of
the Record interface returns a value, it should just automatically be set as a
message property. What
david-streamlio commented on a change in pull request #9943:
URL: https://github.com/apache/pulsar/pull/9943#discussion_r596341448
##
File path:
pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/JavaInstanceRunnable.java
##
@@ -736,8 +737,14 @@ priva
david-streamlio commented on a change in pull request #9943:
URL: https://github.com/apache/pulsar/pull/9943#discussion_r596341448
##
File path:
pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/JavaInstanceRunnable.java
##
@@ -736,8 +737,14 @@ priva
david-streamlio commented on a change in pull request #9943:
URL: https://github.com/apache/pulsar/pull/9943#discussion_r596341448
##
File path:
pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/JavaInstanceRunnable.java
##
@@ -736,8 +737,14 @@ priva
david-streamlio commented on a change in pull request #9943:
URL: https://github.com/apache/pulsar/pull/9943#discussion_r596341448
##
File path:
pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/JavaInstanceRunnable.java
##
@@ -736,8 +737,14 @@ priva
frankjkelly commented on issue #5510:
URL: https://github.com/apache/pulsar/issues/5510#issuecomment-801377287
Thanks @lhotari I have hit this too in Apache Pulsar 2.6.1
CC @addisonj
This is an automated message from the
devinbost opened a new issue #9944:
URL: https://github.com/apache/pulsar/issues/9944
It seems like every once in a while I find myself facing this same issue in
IntelliJ with Pulsar, and I can't ever seem to remember how to get it to go
away...
` is same with
`setBlockIfQueueFull(true)` in C++, or
congbobo184 opened a new pull request #9942:
URL: https://github.com/apache/pulsar/pull/9942
## Motivation
Transaction buffer metrics add.
## implement
```
/** The active transactions. */
public long activeTransactions;
/** The commit transaction count o
michaeljmarshall opened a new pull request #9941:
URL: https://github.com/apache/pulsar/pull/9941
This PR includes basic improvements to table formatting in the
`reference-configuration.md` file. There were several issues:
1. Some rows were missing a pipe (`|`). This didn't affect re
This is an automated email from the ASF dual-hosted git repository.
blankensteiner pushed a change to annotated tag 1.0.0
in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git.
*** WARNING: tag 1.0.0 was modified! ***
from 6950c11 (commit)
to 63f855b (tag)
tagging
This is an automated email from the ASF dual-hosted git repository.
blankensteiner pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 6950c11 Make ready for 1.0.0
gaoran10 commented on a change in pull request #9910:
URL: https://github.com/apache/pulsar/pull/9910#discussion_r596078188
##
File path:
pulsar-sql/presto-pulsar/src/main/java/org/apache/pulsar/sql/presto/PulsarConnectorCache.java
##
@@ -112,7 +112,7 @@ private static Managed
gaoran10 commented on a change in pull request #9910:
URL: https://github.com/apache/pulsar/pull/9910#discussion_r596078188
##
File path:
pulsar-sql/presto-pulsar/src/main/java/org/apache/pulsar/sql/presto/PulsarConnectorCache.java
##
@@ -112,7 +112,7 @@ private static Managed
fransguelinckx opened a new issue #9940:
URL: https://github.com/apache/pulsar/issues/9940
**Describe the bug**
Messages can get consumed out of order with key_shared subscriptions and
small receiverQueueSize
**To Reproduce**
Steps to reproduce the behavior:
* A producer tha
eolivelli commented on pull request #9898:
URL: https://github.com/apache/pulsar/pull/9898#issuecomment-801135722
Avro 1.10.2 has been released today, I will update the patch once CI passes
and I am sure that all tests are passing
--
eolivelli commented on a change in pull request #9590:
URL: https://github.com/apache/pulsar/pull/9590#discussion_r596069056
##
File path:
tests/docker-images/java-test-functions/src/main/java/org/apache/pulsar/tests/integration/io/GenericRecordSource.java
##
@@ -0,0 +1,88 @@
TIBCOeddie opened a new issue #9939:
URL: https://github.com/apache/pulsar/issues/9939
https://github.com/apache/pulsar/blob/94bf13f2f216c9f5d7ca00af31a5efaedde7e397/pulsar-io/influxdb/src/main/java/org/apache/pulsar/io/influxdb/v2/InfluxDBSink.java#L70
---
KannarFr edited a comment on issue #6930:
URL: https://github.com/apache/pulsar/issues/6930#issuecomment-800187333
@pointearth @sijie How do you imagine the index composition regarding
ledgers? As the first implementation, regarding pub/sub system, timestamp
index-based would be a good sta
KannarFr commented on issue #6930:
URL: https://github.com/apache/pulsar/issues/6930#issuecomment-801032939
And where/how we store this index?
This is an automated message from the Apache Git Service.
To respond to the messa
yitian108 commented on issue #9796:
URL: https://github.com/apache/pulsar/issues/9796#issuecomment-801012161
Unluckily, when I try to add the configuration ProducerConfiguration as
above, the segment fault still persist, and a small good news, if I set the
producerConf.setBlockIfQueueFull(
Anonymitaet commented on pull request #9938:
URL: https://github.com/apache/pulsar/pull/9938#issuecomment-800986101
@merlimat @zymap @codelipenghui @Jennifer88huang @Huanli-Meng could you
please help review? Thanks
This is a
Anonymitaet opened a new pull request #9938:
URL: https://github.com/apache/pulsar/pull/9938
Add docs for https://github.com/apache/pulsar/pull/8992
This is an automated message from the Apache Git Service.
To respond to the
Anonymitaet commented on pull request #9937:
URL: https://github.com/apache/pulsar/pull/9937#issuecomment-800979277
@deonvdv @zymap @codelipenghui @Huanli-Meng @Jennifer88huang could you
please help review? Thanks
This is an
Anonymitaet opened a new pull request #9937:
URL: https://github.com/apache/pulsar/pull/9937
Add docs for https://github.com/apache/pulsar/pull/8957.
This is an automated message from the Apache Git Service.
To respond to the
1 - 100 of 113 matches
Mail list logo
