[GitHub] eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

2017-07-27 Thread git
eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#issuecomment-318281496 Closing as won't fix This is an

[GitHub] eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

2017-07-27 Thread git
eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#issuecomment-318271628 @sijie maybe a diffent approach would be to add back the legacy methods with a default implementation

[GitHub] eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

2017-07-27 Thread git
eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#issuecomment-318269931 @sijie sure let's fix the problem for 4.5 doing the best as possible. I do not want large

[GitHub] eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

2017-07-26 Thread git
eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#issuecomment-318244413 @sijie overall I am not very happy with this patch, as I told on the mailing list, addressing the