Andrey Yegorov comments

Results 63 comments of


                                            Andrey Yegorov

[improve] Handle PositionInfo that's too large to serialize as a single entry

@lhotari I got rid of byte[] usage and added a feature flag, per your comments

Add tls config to etcd metadata driver

Please fix checkstyle errors to let CI run

Add tls config to etcd metadata driver

@alesharik please add some documentation, rebase, and make sure that the CI passes

Task: upgrade RocksDB

@bhiwagade-rahul with the minor version upgrade the CI should be good enough; maybe a short perf run. For the major version upgrade we'd want to test with Apache Pulsar, test...

Require Java 17 or Java 21 for building Bookkeeper

I think we should first refactor BK client into a separate module that will be built with JDK 8, then we can move BK server to JDK 17 https://github.com/apache/bookkeeper/issues/617

Do not skip opened ledger in repair not adhering placement ledger

Fencing is not exactly a cheap operation for application using BK. E.g. in case of BK as a WAL it may cause app crash/restart/recovery from WAL etc, in other cases...

Disable annoying logs when segments is empty

re-triggering CI by close/open

[improve] Expose asyncBatchReadUnconfirmed to ReadHandle.

This change requires a BP and discussion/vote in the mail list

[Improve][Client] Group writing into the channel in PerChannelBookieClient

I think this should be configurable. It improves throughput for some workloads, it is possible that more latency-sensitive workloads would want to disable this

Upgrade RocksDB to version 9.2.1

> Have you verified any compatible issues from 6.16.x to 9.2.1 and 7.x to 9.2.1? @hangc0276 I fixed/updated backward compat tests, that's it. If these aren't enough we need to...