Re: BP-54: Repaired the ledger fragment which ensemble not adhere placement policy.

2022-07-29 Thread Yan Zhao
You means we define api in EnsemblePlacementPolicy, and use SPI to load special implementation? On 2022/07/29 15:10:05 Enrico Olivelli wrote: > I support this feature. > > We should design the changes to the APIs. > We need to change EnsemblePlacementPolicy, that is a pluggable API: > users

Re: BP-54: Repaired the ledger fragment which ensemble not adhere placement policy.

2022-07-29 Thread Enrico Olivelli
I support this feature. We should design the changes to the APIs. We need to change EnsemblePlacementPolicy, that is a pluggable API: users can provide. their own implementation please add the design decisions you want to make Thanks Enrico Il giorno ven 29 lug 2022 alle ore 15:31 Yan Zhao

Re: BP-54: Repaired the ledger fragment which ensemble not adhere placement policy.

2022-07-29 Thread Yan Zhao
@Enrico

Re: [DISCUSS] Unify the import order in bookkeeper

2022-07-29 Thread Enrico Olivelli
I am fine with fixing the import order. the patch is still huge, but it is doable I will review the patch and give my feedback thanks Enrico Il giorno ven 29 lug 2022 alle ore 05:48 ZhangJian He ha scritto: > > https://github.com/apache/bookkeeper/pull/3311 is ready for review. > > Thanks >