Re: [DISCUSS][BP-2] Resource aware data placement

2016-12-14 Thread Rithin Shetty
Hi Enrico, Thanks for the comments. Please see my replies inline below: On Wed, Dec 14, 2016 at 4:08 AM, Enrico Olivelli wrote: > Very interesting, > some other notes for the discussion: > - as we are now on java8 can you add a "default" no-op implementation > to the

Re: [DISCUSS][BP-2] Resource aware data placement

2016-12-14 Thread Rithin Shetty
Hi Sijie, See my responses below: On Tue, Dec 13, 2016 at 7:12 PM, Sijie Guo wrote: > Hi Rithin, > > The proposal looks good to me. I have a few comments: > > - How will the BookieInfo structure look like? > I've updated the wiki with this info now. It looks like this:

Build failed in Jenkins: bookkeeper-master-git-pullrequest #172

2016-12-14 Thread Apache Jenkins Server
See Changes: [rithin.shetty] BOOKKEEPER-950: Ledger placement policy to accomodate different storage -- [...truncated 4506 lines...] 136/926 KB 140/926 KB 144/926 KB

[jira] [Commented] (BOOKKEEPER-950) Ledger placement policy to accomodate different storage capacity of bookies

2016-12-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15749274#comment-15749274 ] ASF GitHub Bot commented on BOOKKEEPER-950: --- GitHub user rithin-shetty opened a pull

[jira] [Commented] (BOOKKEEPER-874) Explict LAC from Writer to Bookies

2016-12-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-874?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15749189#comment-15749189 ] ASF GitHub Bot commented on BOOKKEEPER-874: --- Github user sijie commented on the issue:

[jira] [Commented] (BOOKKEEPER-974) make pushing a docker image as part of the release procedure

2016-12-14 Thread Enrico Olivelli (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15748671#comment-15748671 ] Enrico Olivelli commented on BOOKKEEPER-974: I'm trying your image {code} make build

Build failed in Jenkins: bookkeeper-master #1596

2016-12-14 Thread Apache Jenkins Server
See -- [...truncated 369 lines...] [INFO] [INFO] --- maven-resources-plugin:2.4.3:testResources (default-testResources) @ bookkeeper-server --- [INFO] Using 'UTF-8' encoding to copy filtered

Re: [DISCUSS][BP-2] Resource aware data placement

2016-12-14 Thread Enrico Olivelli
Very interesting, some other notes for the discussion: - as we are now on java8 can you add a "default" no-op implementation to the new method in EnsemblePlacementPolicy#updateBookieInfo, this way we will not break compatibility with custom policies - inside the new BookieInfo can we provide a set