rdhabalia commented on PR #3214:
URL: https://github.com/apache/bookkeeper/pull/3214#issuecomment-1211526187
I think the main issue is #2973 implementation. You should not add a
blocking call to throttle unless you have sync API.
The right fix for #2973 is to check the permit and if the p
gaozhangmin opened a new pull request, #3446:
URL: https://github.com/apache/bookkeeper/pull/3446
### Motivation
During autoRecovery, there were so many `Bookie handle is not available`
errors, it was because replicateWorker would tried to read entry from bookie
even if it's alrea
gaozhangmin closed pull request #3446: Support skipping unavailable bookie when
reading entry
URL: https://github.com/apache/bookkeeper/pull/3446
--
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
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1211856787
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
gaozhangmin closed pull request #3446: Support skipping unavailable bookie when
reading entry
URL: https://github.com/apache/bookkeeper/pull/3446
--
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
hangc0276 commented on issue #3445:
URL: https://github.com/apache/bookkeeper/issues/3445#issuecomment-1212212437
It looks like the ledgerId is < 0, which will lead to the above exception. I
wonder whether your ledger log file is broken.
https://github.com/apache/bookkeeper/blob/de5d45fb
lhotari commented on PR #3444:
URL: https://github.com/apache/bookkeeper/pull/3444#issuecomment-1212256336
> We already have the index lookup stats `read-locations-index-time`, does
it has any case this metric can't cover?
>
>
https://github.com/apache/bookkeeper/blob/de5d45fb823dc55
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1212649794
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
gaozhangmin closed pull request #3446: Support reading entry fail-fast on
unavailable bookie
URL: https://github.com/apache/bookkeeper/pull/3446
--
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
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1214249759
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
wangjialing218 commented on code in PR #3158:
URL: https://github.com/apache/bookkeeper/pull/3158#discussion_r945371704
##
bookkeeper-common/src/test/java/org/apache/bookkeeper/common/net/ServiceURITest.java:
##
@@ -206,6 +206,30 @@ public void testServiceInfoPlus() {
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1214822071
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
disserakt commented on issue #3445:
URL: https://github.com/apache/bookkeeper/issues/3445#issuecomment-1215270686
@hangc0276 - How can I check if the ledger log file is broken or not?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216040104
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216187201
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216305868
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216355600
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
eolivelli commented on PR #3443:
URL: https://github.com/apache/bookkeeper/pull/3443#issuecomment-1216461826
@nicoloboschi @michaeljmarshall @zymap @merlimat
TLS tests are failing, this is unrelated to this change.
I am committing the patch
--
This is an automated message from t
eolivelli merged PR #3443:
URL: https://github.com/apache/bookkeeper/pull/3443
--
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: issues-unsubscr...@bookke
eolivelli merged PR #3444:
URL: https://github.com/apache/bookkeeper/pull/3444
--
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: issues-unsubscr...@bookke
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216575392
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1216796484
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217358847
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217409959
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217421099
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217459504
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217501028
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT opened a new issue, #3447:
URL: https://github.com/apache/bookkeeper/issues/3447
TestTLS.testConnectToLocalTLSClusterTLSClient can't run all successfully in
branch master and 4.15.1-rc0,
it seems that this problem has appeared long ago, rather than being
introduced separately
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217554227
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1217945930
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
zymap commented on issue #3447:
URL: https://github.com/apache/bookkeeper/issues/3447#issuecomment-1218827844
https://github.com/apache/bookkeeper/pull/3439
this PR is for this
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
zymap opened a new pull request, #3448:
URL: https://github.com/apache/bookkeeper/pull/3448
---
*Motivation*
When runing the tls test, the remote address is a LocalAddress,
but we cast it to the `InetSocketAddress`. That cause the test
`testConnectToLocalTLSClusterTLSClien
zymap commented on PR #3448:
URL: https://github.com/apache/bookkeeper/pull/3448#issuecomment-1219349641
Now the TLS test has no failures
https://pipelines.actions.githubusercontent.com/serviceHosts/18fb66ef-1cbe-4dfc-9200-fca890f3f8b2/_apis/pipelines/1/runs/42799/signedlogcontent/2?urlE
eolivelli merged PR #3448:
URL: https://github.com/apache/bookkeeper/pull/3448
--
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: issues-unsubscr...@bookke
eolivelli commented on PR #3448:
URL: https://github.com/apache/bookkeeper/pull/3448#issuecomment-1219385238
@zymap please cherry-pick to 4.15 and 4.14 branches
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
zymap commented on PR #3448:
URL: https://github.com/apache/bookkeeper/pull/3448#issuecomment-1219413551
Sure
--
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
tisonkun opened a new pull request, #3449:
URL: https://github.com/apache/bookkeeper/pull/3449
Reference:
* https://github.com/apache/bookkeeper/pull/2432
* https://github.com/apache/pulsar/issues/7477
* https://github.com/grpc/grpc/pull/25418
*
https://github.com/apache/puls
eolivelli commented on code in PR #3449:
URL: https://github.com/apache/bookkeeper/pull/3449#discussion_r949422949
##
stream/clients/python/setup.py:
##
@@ -31,7 +31,7 @@
'six>=1.10.0',
'pytz',
'futures>=3.2.0;python_version<"3.2"',
-'grpcio<1.28,>=1.8.2',
+
zymap commented on PR #3448:
URL: https://github.com/apache/bookkeeper/pull/3448#issuecomment-1220118528
cherry-picked to 4.14 and 4.15
--
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 specifi
tisonkun commented on code in PR #3449:
URL: https://github.com/apache/bookkeeper/pull/3449#discussion_r949717085
##
stream/clients/python/setup.py:
##
@@ -31,7 +31,7 @@
'six>=1.10.0',
'pytz',
'futures>=3.2.0;python_version<"3.2"',
-'grpcio<1.28,>=1.8.2',
+
tisonkun closed pull request #3449: Bump grpcio dep version of Python client
for supporting ARM
URL: https://github.com/apache/bookkeeper/pull/3449
--
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 th
tisonkun commented on PR #3449:
URL: https://github.com/apache/bookkeeper/pull/3449#issuecomment-1220133096
Closed for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
hangc0276 opened a new pull request, #3450:
URL: https://github.com/apache/bookkeeper/pull/3450
### Modification
Fix the wrong `cbThreadPoolQueueSize` doc name
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
zymap commented on code in PR #3450:
URL: https://github.com/apache/bookkeeper/pull/3450#discussion_r949742882
##
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/stats/JournalStats.java:
##
@@ -216,7 +217,7 @@ public JournalStats(StatsLogger statsLogger, final long
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1220348074
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
eolivelli commented on code in PR #3368:
URL: https://github.com/apache/bookkeeper/pull/3368#discussion_r949895963
##
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LedgerMetadataIndexAsync.java:
##
@@ -0,0 +1,117 @@
+/**
+ *
+ * Licensed to the Apache
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1220383976
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on code in PR #3368:
URL: https://github.com/apache/bookkeeper/pull/3368#discussion_r949975958
##
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LedgerMetadataIndexAsync.java:
##
@@ -0,0 +1,117 @@
+/**
+ *
+ * Licensed to the Apache
hangc0276 closed pull request #3450: Fix wrong stats name
URL: https://github.com/apache/bookkeeper/pull/3450
--
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-
StevenLuMT commented on PR #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#issuecomment-1220852299
rerun failure checks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
StevenLuMT commented on issue #3447:
URL: https://github.com/apache/bookkeeper/issues/3447#issuecomment-1220881695
>
yes, look good to me,thanks @zymap
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
eolivelli merged PR #3439:
URL: https://github.com/apache/bookkeeper/pull/3439
--
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: issues-unsubscr...@bookke
eolivelli commented on issue #2205: Ledger having a Empty segment is causing
ReplicasCheck scrutiny to fail
URL: https://github.com/apache/bookkeeper/pull/2205#issuecomment-557461579
run bookkeeper-server client tests
This is
eolivelli closed pull request #2164: Prepare website for 4.10.0 release
URL: https://github.com/apache/bookkeeper/pull/2164
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
eolivelli commented on issue #2204: Bump jackson.version from 2.9.7 to 2.10.1
URL: https://github.com/apache/bookkeeper/pull/2204#issuecomment-557462035
We have to fix the license files
com.fasterxml.jackson.core-jackson-annotations-2.10.1.jar unaccounted for in
LICENSE
com.faster
eolivelli commented on issue #2182: Add API for Bookie checksum verification
writeFlags
URL: https://github.com/apache/bookkeeper/pull/2182#issuecomment-557462688
@karanmehta93
How are you doing with this change ?
This is
karanmehta93 commented on issue #2182: Add API for Bookie checksum verification
writeFlags
URL: https://github.com/apache/bookkeeper/pull/2182#issuecomment-557823867
@eolivelli Currently I am not working, busy with some internal stuff.
Will update the PR when I get some time.
-
Ghatage commented on issue #2194: Creating namespace throws error
URL: https://github.com/apache/bookkeeper/issues/2194#issuecomment-559332446
@eolivelli should we close this?
This is an automated message from the Apache Git S
Ghatage commented on issue #2140: how does the client choose the bookie to read?
URL: https://github.com/apache/bookkeeper/issues/2140#issuecomment-559332658
@eolivelli @sijie should we close this?
This is an automated message
sunjingji closed issue #2140: how does the client choose the bookie to read?
URL: https://github.com/apache/bookkeeper/issues/2140
This is an automated message from the Apache Git Service.
To respond to the message, please lo
codelipenghui opened a new issue #2208: NPE in
RackwareEnsemblePlacementPolicyImpl.
URL: https://github.com/apache/bookkeeper/issues/2208
**Describe the bug**
```
java.lang.NullPointerException: null
at
org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl.isEnsem
eolivelli commented on issue #2204: Bump jackson.version from 2.9.7 to 2.10.1
URL: https://github.com/apache/bookkeeper/pull/2204#issuecomment-560557891
@rdhabalia or @dmercuriali does any of you have cycles to pick up this
issue ?
-
eolivelli commented on a change in pull request #2203: Issue #609 Add indexDirs
field to cookie
URL: https://github.com/apache/bookkeeper/pull/2203#discussion_r352833467
##
File path:
tools/ledger/src/test/java/org/apache/bookkeeper/tools/cli/commands/cookie/GetCookieCommandTest.ja
eolivelli commented on a change in pull request #2203: Issue #609 Add indexDirs
field to cookie
URL: https://github.com/apache/bookkeeper/pull/2203#discussion_r352836177
##
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Cookie.java
##
@@ -124,14 +
Ghatage commented on a change in pull request #2203: Issue #609 Add indexDirs
field to cookie
URL: https://github.com/apache/bookkeeper/pull/2203#discussion_r352885205
##
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Cookie.java
##
@@ -124,14 +12
Ghatage commented on a change in pull request #2203: Issue #609 Add indexDirs
field to cookie
URL: https://github.com/apache/bookkeeper/pull/2203#discussion_r352885472
##
File path:
tools/ledger/src/test/java/org/apache/bookkeeper/tools/cli/commands/cookie/GetCookieCommandTest.java
Ghatage opened a new issue #2209: BP-40 Audit logging for BookieShell
URL: https://github.com/apache/bookkeeper/issues/2209
**BP**
This is the master ticket for tracking BP-40 : Audit logging for BookieShell.
In order to make a software compliant with the security and privacy
SunDapeng1 opened a new pull request #2210: Fixed bin/bookkeeper shell
readjournal
URL: https://github.com/apache/bookkeeper/pull/2210
### Motivation
An error occurred while executing `bin/bookkeeper shell readjournal`,
.
When I activate the Strea
eolivelli commented on issue #2216: Bookkeeper High CPU usage when
StreamStorageLifecycleComponent extra component is active
URL: https://github.com/apache/bookkeeper/issues/2216#issuecomment-565433414
Are you able to take a dump of the stacktrace of the JVM? With jstack for
instance
aschiazza commented on issue #2216: Bookkeeper High CPU usage when
StreamStorageLifecycleComponent extra component is active
URL: https://github.com/apache/bookkeeper/issues/2216#issuecomment-565438568
Thanks for the reply.
I have attached the JVM stacktrace and ps output.
[j
eolivelli commented on issue #2216: Bookkeeper High CPU usage when
StreamStorageLifecycleComponent extra component is active
URL: https://github.com/apache/bookkeeper/issues/2216#issuecomment-565448669
It looks like a problem with some leader electiin.
I can't tell more now (I don't have
aschiazza commented on issue #2216: Bookkeeper High CPU usage when
StreamStorageLifecycleComponent extra component is active
URL: https://github.com/apache/bookkeeper/issues/2216#issuecomment-565456508
I think bookie is able to talk to zookeeper. I test it in that way:
1- I attached a sh
aschiazza edited a comment on issue #2216: Bookkeeper High CPU usage when
StreamStorageLifecycleComponent extra component is active
URL: https://github.com/apache/bookkeeper/issues/2216#issuecomment-565456508
I think bookie is able to talk to zookeeper. I test it in that way:
1- I attach
Anonymitaet commented on issue #2193: [Doc] Update BookKeeper Guide
URL: https://github.com/apache/bookkeeper/issues/2193#issuecomment-565701284
I divide this task into several subtasks and add a plan.
### Get started
- [ ] Run bookie locally (2019/12)
- [ ] Run bookie in Do
Anonymitaet edited a comment on issue #2193: [Doc] Update BookKeeper Guide
URL: https://github.com/apache/bookkeeper/issues/2193#issuecomment-565701284
I divide this task into several subtasks and add a plan.
### Get started
- [ ] Run bookie locally (2019/12)
- [ ] Run bookie in
Anonymitaet opened a new pull request #2217: [Doc]Update Get Started Guide
URL: https://github.com/apache/bookkeeper/pull/2217
Fix a part of https://github.com/apache/bookkeeper/issues/2193.
I've updated this chapter based on the following architecture.
- [ ] Run bookie i
Anonymitaet commented on issue #2217: [Doc]Update Get Started Guide
URL: https://github.com/apache/bookkeeper/pull/2217#issuecomment-566361982
@sijie could you please help review? Thank you
This is an automated message from th
eolivelli commented on a change in pull request #2217: [Doc]Update Get Started
Guide
URL: https://github.com/apache/bookkeeper/pull/2217#discussion_r358622821
##
File path: site/docs/latest/getting-started/run-locally.md
##
@@ -1,16 +1,115 @@
---
-title: Run bookies local
eolivelli opened a new issue #2218: Write customMetadata on ledgers created by
DistributedLog
URL: https://github.com/apache/bookkeeper/issues/2218
**FEATURE REQUEST**
1. Please describe the feature you are requesting.
It is very common for BookKeeper based applications to tag led
eolivelli commented on issue #2218: Write customMetadata on ledgers created by
DistributedLog
URL: https://github.com/apache/bookkeeper/issues/2218#issuecomment-566429132
@diegosalvi @dmercuriali PTAL
This is an automated mes
ravisharda opened a new pull request #2219: Fix issues in Docker image that
caused failures upon container startup
URL: https://github.com/apache/bookkeeper/pull/2219
Descriptions of the changes in this PR:
Fixes issues in Docker image that were causing errors when containers were
spun.
eolivelli commented on issue #2219: Issue #2212: Fix issues in Docker image
that caused failures upon container startup
URL: https://github.com/apache/bookkeeper/pull/2219#issuecomment-566471970
@dmercuriali PTAL
This is an a
nicknezis commented on issue #2219: Issue #2212: Fix issues in Bookkeeper
Docker image that prevented containers from starting up
URL: https://github.com/apache/bookkeeper/pull/2219#issuecomment-566847033
I think this might fix issues I've had when trying to run Bookkeeper in
Kubernetes.
nicknezis commented on issue #2219: Issue #2212: Fix issues in Bookkeeper
Docker image that prevented containers from starting up
URL: https://github.com/apache/bookkeeper/pull/2219#issuecomment-566863256
Ran a local test with Heron Helm chart in Minikube using a locally built
copy of this
andrekramer1 opened a new issue #2220: Exception while replaying journals,
shutting down
URL: https://github.com/apache/bookkeeper/issues/2220
We're running Apache Pulsar with bookkeeper on a Kubernetes cluster and we
have bookies (2/3) failing to restart with the following exception:
0
eolivelli commented on issue #2211: bin/bookkeeper shell supports the -conf
URL: https://github.com/apache/bookkeeper/pull/2211#issuecomment-568245699
run integration tests
This is an automated message from the Apache Git Serv
eolivelli commented on issue #2210: Fixed bin/bookkeeper shell readjournal
URL: https://github.com/apache/bookkeeper/pull/2210#issuecomment-568245726
run bookkeeper-server replication tests
This is an automated message from t
eolivelli opened a new issue #2221: Add support for https in Vertx HTTP server
URL: https://github.com/apache/bookkeeper/issues/2221
**FEATURE REQUEST**
1. Please describe the feature you are requesting.
Currently default http server (Vertx) do not start an https endpoint, we
1 - 100 of 22798 matches
Mail list logo
