Hi Karl, I don't see think this is a problem. discovery.base is meant to be a 'base bundle' that other discovery implementors can use (namely at the moment in Sling: discovery.oak and discovery.impl) if they wish. In that sense its an SPI that we should be free to change. The real API of discovery is discovery.api, and that remains unchanged. So I think this is fine.
Cheers, Stefan On 08/05/17 00:12, "Karl Pauls" <karlpa...@gmail.com> wrote: >Hi, > >while trying to get the discovery artifacts impacted by the json >removal ready for release I realized that we are about to increase the >version of: > >org.apache.sling.discovery.base.connectors.announcement > >from 1.1.2 to 2.0.0 because that is another place where the JSONObject >is part of the api. The culprit in this case is the Announcement >class. > >What do people think, is this a problem and if so what should we do about >it? > >regards, > >Karl > > >-- >Karl Pauls >karlpa...@gmail.com
