I think this is right update for the API signatures to not require Guava dependency.
Breaking APIs is usually preferable early, especially when we are still in incubator. Kind of like "pardon the interruption" card we could use for Apache projects in incubation :) But in seriousness, this update would be beneficial in the long run. - Henry On Tue, May 5, 2015 at 4:07 PM, Terence Yim <[email protected]> wrote: > Hi all, > > I try to address TWILL-69 (Remove guava dependency from twill-api and > twill-common modules) and would like to bring up the discussion about > having a backward incompatible change in 0.6.0 release. > > Removing guava will impact TwillController and TwillRunnerService, > which both interfaces currently extends from guava Service and are in > twilll-api. Do you guys think it's ok to break backward compatibility > in 0.6.0? Or any suggestions on how to do it without break > compatibility? > > Currently I have a backward incompatible change that is posted in PR > https://github.com/apache/incubator-twill/pull/32 > > Thanks, > Terence
