Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-27 Thread Enrico Olivelli
The end of the story is that discussing with Sijie we decided to close the issue as won't fix. In the future we will define the rules to handle backward compatibility. Changes on the PlacementPolicy are about semantics from 4.4 to 4.5 Enrico Il mer 26 lug 2017, 12:26 Enrico Olivelli

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-25 Thread Enrico Olivelli
Il mar 25 lug 2017, 19:08 Sijie Guo ha scritto: > On Tue, Jul 25, 2017 at 9:24 AM, Enrico Olivelli > wrote: > > > Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > > > > > Let me try to summarize the comments here (also as a

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-25 Thread Sijie Guo
On Tue, Jul 25, 2017 at 9:24 AM, Enrico Olivelli wrote: > Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > > > Let me try to summarize the comments here (also as a reference for other > > people). This basically is comprised of two questions: > > > > 1)

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-25 Thread Enrico Olivelli
Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > Let me try to summarize the comments here (also as a reference for other > people). This basically is comprised of two questions: > > 1) how do we handle the 'binary backward compatibility' for drop-in upgrade > in

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-25 Thread Sijie Guo
Let me try to summarize the comments here (also as a reference for other people). This basically is comprised of two questions: 1) how do we handle the 'binary backward compatibility' for drop-in upgrade in applications using only one bk version? ideally if we change any method signatures in

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-25 Thread Enrico Olivelli
2017-07-24 22:54 GMT+02:00 Sijie Guo : > On Mon, Jul 24, 2017 at 1:29 PM, Enrico Olivelli > wrote: > > > Il lun 24 lug 2017, 19:39 Sijie Guo ha scritto: > > > > > On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-24 Thread Sijie Guo
On Mon, Jul 24, 2017 at 1:29 PM, Enrico Olivelli wrote: > Il lun 24 lug 2017, 19:39 Sijie Guo ha scritto: > > > On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli > > wrote: > > > > > Hi, > > > we are going to release 4.5 soon, in

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-24 Thread Enrico Olivelli
Il lun 24 lug 2017, 19:39 Sijie Guo ha scritto: > On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli > wrote: > > > Hi, > > we are going to release 4.5 soon, in this release we changed > > EnsemblePlacementPolicy interface. > > > > It you are using a

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-24 Thread Sijie Guo
On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli wrote: > Hi, > we are going to release 4.5 soon, in this release we changed > EnsemblePlacementPolicy interface. > > It you are using a "custom" EnsemblePlacementPolicy in 4.4 and you just > switch the bookeeeper-server JAR on

EnsemblePlacementPolicy Public API breaking changes

2017-07-24 Thread Enrico Olivelli
Hi, we are going to release 4.5 soon, in this release we changed EnsemblePlacementPolicy interface. It you are using a "custom" EnsemblePlacementPolicy in 4.4 and you just switch the bookeeeper-server JAR on the classpath your application won't run. I already got that problem and I needed to