jai1 commented on a change in pull request #1451: Fix: NPE on http-lookup
redirection
URL: https://github.com/apache/incubator-pulsar/pull/1451#discussion_r177314778
##
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java
##
@@
merlimat commented on issue #1449: When running standalone broker, force the
proxy redirection flag
URL: https://github.com/apache/incubator-pulsar/pull/1449#issuecomment-376403603
retest this please
This is an automated
merlimat commented on issue #1451: Fix: NPE on http-lookup redirection
URL: https://github.com/apache/incubator-pulsar/pull/1451#issuecomment-376403137
retest this please
This is an automated message from the Apache Git
merlimat commented on issue #1451: Fix: NPE on http-lookup redirection
URL: https://github.com/apache/incubator-pulsar/pull/1451#issuecomment-376403137
retest this please
This is an automated message from the Apache Git
merlimat commented on issue #1410: Convert admin client properties to use v2
endpoints.
URL: https://github.com/apache/incubator-pulsar/pull/1410#issuecomment-376396644
retest this please
This is an automated message from
merlimat commented on issue #1445: Add version warning for latest vs. 2.0
URL: https://github.com/apache/incubator-pulsar/pull/1445#issuecomment-376396537
retest this please
This is an automated message from the Apache Git
merlimat commented on issue #1445: Add version warning for latest vs. 2.0
URL: https://github.com/apache/incubator-pulsar/pull/1445#issuecomment-376396537
retest this please
This is an automated message from the Apache Git
zhaijack commented on issue #1428: Issue #1117: handle race in concurrent
bundle split
URL: https://github.com/apache/incubator-pulsar/pull/1428#issuecomment-376395978
@merlimat , test fail because of wrongly removed `getData` in
LocalZooKeeperCacheService in commit
zhaijack commented on issue #1428: Issue #1117: handle race in concurrent
bundle split
URL: https://github.com/apache/incubator-pulsar/pull/1428#issuecomment-376395978
@merlimat , test fail because of wrongly removed `getData` in
LocalZooKeeperCacheService in commit
zhaijack commented on a change in pull request #1428: Issue #1117: handle race
in concurrent bundle split
URL: https://github.com/apache/incubator-pulsar/pull/1428#discussion_r177306959
##
File path:
zubchenok opened a new issue #1452: reachedEndOfTopic is called twice if a
topic has been terminated before subscription
URL: https://github.com/apache/incubator-pulsar/issues/1452
When you terminate a topic and then subscribe,
MessageListener.reachedEndOfTopic is called two times.
rdhabalia opened a new pull request #1451: Fix: NPE on http-lookup redirection
URL: https://github.com/apache/incubator-pulsar/pull/1451
### Motivation
- If broker has TLS enabled and broker's TLS url is not setup then
http-redirection throws NPE.
- Also, broker should consider
merlimat opened a new pull request #1450: Added multiple position delete in
ManagedLedger
URL: https://github.com/apache/incubator-pulsar/pull/1450
### Motivation
To support the grouping of acknowledgements, we need to be able to pass
multiple message ids to managed ledger to
zhaijack commented on issue #1428: Issue #1117: handle race in concurrent
bundle split
URL: https://github.com/apache/incubator-pulsar/pull/1428#issuecomment-376367021
@merlimat thanks, will handle it now
This is an
merlimat commented on issue #1428: Issue #1117: handle race in concurrent
bundle split
URL: https://github.com/apache/incubator-pulsar/pull/1428#issuecomment-376362244
@zhaijack Please merge with master that has fix for a very flaky test.
Also it appears there's a test that might
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new b1fbd3a Extend Spark Streaming
merlimat closed pull request #1448: [Spark_Streaming] Extend Receiver UT
Coverage
URL: https://github.com/apache/incubator-pulsar/pull/1448
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this
sijie commented on issue #1424: Improve batch message acking by removing batch
message tracker
URL: https://github.com/apache/incubator-pulsar/pull/1424#issuecomment-376352303
merged latest master
This is an automated
sijie commented on a change in pull request #1447: Send Log statements to log
Topic for Java Functions
URL: https://github.com/apache/incubator-pulsar/pull/1447#discussion_r177271917
##
File path:
sijie commented on a change in pull request #1441: Update the bookkeeper
configuration file
URL: https://github.com/apache/incubator-pulsar/pull/1441#discussion_r177271361
##
File path: conf/bookkeeper.conf
##
@@ -39,101 +68,231 @@ bookiePort=3181
# set the listening
merlimat commented on issue #1424: Improve batch message acking by removing
batch message tracker
URL: https://github.com/apache/incubator-pulsar/pull/1424#issuecomment-376349548
@sijie can you merge with master. It already has the flaky test fixed.
merlimat commented on issue #1424: Improve batch message acking by removing
batch message tracker
URL: https://github.com/apache/incubator-pulsar/pull/1424#issuecomment-376349548
@sijie can you merge with master? It already has the flaky test fixed.
merlimat opened a new pull request #1449: When running standalone broker, force
the proxy redirection flag
URL: https://github.com/apache/incubator-pulsar/pull/1449
### Motivation
Since standalone is running a "real" Pulsar service (along with ZK and
BookKeeper) it uses the same
erenavsarogullari commented on issue #1448: [Spark_Streaming] Extend Receiver
UT Coverage
URL: https://github.com/apache/incubator-pulsar/pull/1448#issuecomment-376344347
retest this please
This is an automated message from
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new dafe123 Enforce timeout on all
merlimat closed pull request #1347: Enforce timeout on all TestNG tests, thread
dump on timeout
URL: https://github.com/apache/incubator-pulsar/pull/1347
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 01a77aa Avoid allocation of one
merlimat commented on issue #1417: Replace cached data key from the map if
present already
URL: https://github.com/apache/incubator-pulsar/pull/1417#issuecomment-376316438
retest this please
This is an automated message from
merlimat closed pull request #1439: Avoid allocation of one PositionImpl object
in write path
URL: https://github.com/apache/incubator-pulsar/pull/1439
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of
srkukarni commented on issue #1447: Send Log statements to log Topic for Java
Functions
URL: https://github.com/apache/incubator-pulsar/pull/1447#issuecomment-376329446
retest this please
This is an automated message from
erenavsarogullari opened a new pull request #1448: [Spark_Streaming] Extend
Receiver UT Coverage
URL: https://github.com/apache/incubator-pulsar/pull/1448
### Motivation
Extending current `SparkStreamingPulsarReceiver` UT coverage
- Default Settings
- Negative cases (if
merlimat commented on issue #1417: Replace cached data key from the map if
present already
URL: https://github.com/apache/incubator-pulsar/pull/1417#issuecomment-376316438
retest this please
This is an automated message from
merlimat commented on issue #1362: More Pulsar Functions documentation
URL: https://github.com/apache/incubator-pulsar/pull/1362#issuecomment-376315309
retest this please
This is an automated message from the Apache Git
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 02fede7 Added Groovy DSL definition
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a change to branch pr-jobs
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git.
was 46efedc Also run integration test concurrently
The revisions that were on this branch are still
merlimat closed pull request #1444: Added Groovy DSL definition for C++/Java PR
builds
URL: https://github.com/apache/incubator-pulsar/pull/1444
This is an automated message from the Apache Git Service.
To respond to the
merlimat closed pull request #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 358bbc2 Revert "Recycle OpAddEntry
srkukarni commented on issue #1447: Send Log statements to log Topic for Java
Functions
URL: https://github.com/apache/incubator-pulsar/pull/1447#issuecomment-376304711
@sijie @merlimat
This is an automated message from the
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch pr-jobs
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/pr-jobs by this push:
new 46efedc Also run integration test
merlimat commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376296710
retest this please
This is an automated message
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a change to branch pr-jobs
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git.
from 5e6bee7 Added license check
add fa2347f Run PR builds in parallel
No new revisions were added by
sijie commented on a change in pull request #1441: Update the bookkeeper
configuration file
URL: https://github.com/apache/incubator-pulsar/pull/1441#discussion_r177213493
##
File path: conf/bookkeeper.conf
##
@@ -39,101 +68,231 @@ bookiePort=3181
# set the listening
lucperkins commented on issue #1446: Pulsar 2.0 Documentation
URL:
https://github.com/apache/incubator-pulsar/issues/1446#issuecomment-376286514
- [ ] Message deduplication
- [ ] Add "cookbooks" section with end-to-end instructions for specific
tasks (like adding and removing nodes)
sijie commented on issue #1446: Pulsar 2.0 Documentation
URL:
https://github.com/apache/incubator-pulsar/issues/1446#issuecomment-376287138
@lucperkins added "message deduplication"
This is an automated message from the
lucperkins commented on issue #1446: Pulsar 2.0 Documentation
URL:
https://github.com/apache/incubator-pulsar/issues/1446#issuecomment-376286514
- [ ] Message deduplication
This is an automated message from the Apache Git
sijie opened a new issue #1446: Pulsar 2.0 Documentation
URL: https://github.com/apache/incubator-pulsar/issues/1446
Created a master issue for tracking all the documentation tasks for pulsar
2.0.
This is an automated
merlimat commented on issue #1444: Added Groovy DSL definition for C++/Java PR
builds
URL: https://github.com/apache/incubator-pulsar/pull/1444#issuecomment-376284665
retest this please
This is an automated message from the
merlimat commented on issue #1444: Added Groovy DSL definition for C++/Java PR
builds
URL: https://github.com/apache/incubator-pulsar/pull/1444#issuecomment-376284665
retest this please
This is an automated message from the
lucperkins opened a new pull request #1445: Add version warning for latest vs.
2.0
URL: https://github.com/apache/incubator-pulsar/pull/1445
This is an automated message from the Apache Git Service.
To respond to the
merlimat commented on a change in pull request #1441: Update the bookkeeper
configuration file
URL: https://github.com/apache/incubator-pulsar/pull/1441#discussion_r177199255
##
File path: conf/bookkeeper.conf
##
@@ -39,101 +68,231 @@ bookiePort=3181
# set the listening
merlimat commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376273012
retest this please
This is an automated message
merlimat commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376273012
retest this please
This is an automated message
merlimat commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376271412
@rdhabalia I verified that test stops failing after removing the recycle.
merlimat commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376266483
@rdhabalia I think this is probably what's happening:
```
18:19:24.743
sijie commented on issue #1424: Improve batch message acking by removing batch
message tracker
URL: https://github.com/apache/incubator-pulsar/pull/1424#issuecomment-376262456
retest this please
This is an automated message
sijie commented on issue #1440: Update default values for a few publisher
settings
URL: https://github.com/apache/incubator-pulsar/pull/1440#issuecomment-376261949
retest this please
This is an automated message from the
sijie commented on issue #1441: Update the bookkeeper configuration file
URL: https://github.com/apache/incubator-pulsar/pull/1441#issuecomment-376261402
retest this please
This is an automated message from the Apache Git
merlimat opened a new pull request #1444: Added Groovy DSL definition for
C++/Java PR builds
URL: https://github.com/apache/incubator-pulsar/pull/1444
### Motivation
Added Jenkins DSL definitions for new PR validation jobs.
This will create one job for Java8 tests and one for
saandrews commented on issue #1417: Replace cached data key from the map if
present already
URL: https://github.com/apache/incubator-pulsar/pull/1417#issuecomment-376242499
retest this please
This is an automated message
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 3360b14 Fixed C++ formatting (#1443)
merlimat closed pull request #1443: Fixed C++ formatting
URL: https://github.com/apache/incubator-pulsar/pull/1443
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request
rdhabalia commented on issue #1442: Revert "Recycle OpAddEntry when ledger-ops
fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442#issuecomment-376236794
> the callback is calling into a dead object
It seems, `cb.addFailed(e, ctx)` is being triggered before it
merlimat opened a new pull request #1443: Fixed C++ formatting
URL: https://github.com/apache/incubator-pulsar/pull/1443
### Motivation
Recent commit merged broke the formatting.
This is an automated message from the
ivankelly opened a new pull request #1442: Revert "Recycle OpAddEntry when
ledger-ops fails (#1415)"
URL: https://github.com/apache/incubator-pulsar/pull/1442
If the OpAddEntry is recycled before the callback is triggered, the callback
is calling into a dead object.
This reverts
ivankelly commented on issue #1415: Recycle OpAddEntry when ledger-ops fails
URL: https://github.com/apache/incubator-pulsar/pull/1415#issuecomment-376201587
@rdhabalia @merlimat This change is causing flake in
ManagedLedgerBkTest#managedLedgerClosed
If you close the ledger, the op
maskit commented on a change in pull request #1297: Add Configuration to set
tlsClientAuth
URL: https://github.com/apache/incubator-pulsar/pull/1297#discussion_r177043153
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/client/api/TlsProducerConsumerBase.java
jai1 commented on a change in pull request #1297: Add Configuration to set
tlsClientAuth
URL: https://github.com/apache/incubator-pulsar/pull/1297#discussion_r177008602
##
File path:
pulsar-broker/src/test/java/org/apache/pulsar/client/api/TlsProducerConsumerBase.java
##
sijie opened a new pull request #1441: Update the bookkeeper configuration file
URL: https://github.com/apache/incubator-pulsar/pull/1441
Update the bookkeeper configuration file to use the configuration file in
bookkeeper (so that it includes all new settings available in 4.7).
jai1 commented on issue #1297: Add Configuration to set tlsClientAuth
URL: https://github.com/apache/incubator-pulsar/pull/1297#issuecomment-376057341
retest this please
This is an automated message from the Apache
70 matches
Mail list logo