I think we have to stick to the OSGI spec and use unsetXXX and not using resetXXX. (112.8.1 Component Annotations) @Reto Gmür <[email protected]>, you can update the module as suggested. And yes, I think it needs an argument.
Thanks Hasan On Mon, Mar 18, 2019 at 11:51 AM Reto Gmür <[email protected]> wrote: > I looked at the OSGi DS spec section 112.3.2 and it says "An event method > can take one or more parameters." - Which is quite dull given their example > just ignores the parameter: void unsetLog( LoggerFactory l ) { lf = null; } > > Reto > > -----Original Message----- > From: Hasan <[email protected]> > Sent: Monday, March 18, 2019 6:24 AM > To: [email protected] > Subject: Re: [DISCUSS] Release the reunited branch as the master branch > version 8 > > Hi Reto, all, > > On Thu, Mar 14, 2019 at 2:22 PM Reto Gmür <[email protected]> wrote: > > > Hi Hasan > > > > Yes, the renaming you propose makes sense to me. Are both names > > supported by the plugin generating the files in OSGI-INF or is some > > special annotation needed? Does the method need to have an argument? > > > > Sorry, I don't know. Maybe you can try or check? > > Hasan > > > > Cheers, > > Reto > > > > -----Original Message----- > > From: Hasan <[email protected]> > > Sent: Wednesday, March 13, 2019 8:28 PM > > To: [email protected] > > Subject: Re: [DISCUSS] Release the reunited branch as the master > > branch version 8 > > > > Hi Reto > > > > Without a serializer instance, a GraphWriter will have no use. Thus, I > > don't see a disadvantage of the code proposed. > > Not sure though whether the name unsetSerializer should be replaced > > with resetSerializer which would be more suitable in this case, > > because we are going to reset it to a default serializer rather than set > it to null. > > > > Cheers > > Hasan > > > > On Mon, Mar 11, 2019 at 10:19 AM Reto Gmür <[email protected]> > wrote: > > > > > Hi Hasan > > > > > > Regarding GraphWriter: I created a branch reunited-non-ds-mbw to > > > illustrate what I mean. Do you see a disadvantage with that code > > proposal? > > > The same could be applied to GraphReader. > > > > > > Cheers, > > > Reto > > > > > > -----Original Message----- > > > From: Hasan <[email protected]> > > > Sent: Sunday, March 10, 2019 3:59 PM > > > To: [email protected] > > > Subject: Re: [DISCUSS] Release the reunited branch as the master > > > branch version 8 > > > > > > Hi Reto > > > > > > Thx for the input. > > > Comments inline > > > > > > On Sat, Mar 9, 2019 at 5:36 PM Reto Gmür <[email protected]> > wrote: > > > > > > > Hi Hasan, all, > > > > > > > > I've just created a branch of the jena.* modules on > > > > https://github.com/clerezza and of > > > > https://github.com/linked-solutions/slds. > > > > > > > Great. > > > I assume we will fix the module version in the reunited branch to > 2.0.0. > > > Before that I'd like to have the parent version be changed from > > > 8-SNAPSHOT to 8. Do you agree? > > > What can we remove from the parent pom.xml and what can we update ? > > > > > > > > > > > The refactoring brings some good improvements so I'd like to see > > > > this released as soon as possible. > > > > > > > > > > This also means to make this the new master. > > > > > > > > > > In my opinion discussing the following points should nor block the > > > > release, but maybe we find a consensus easily so that we could > > > > incorporate this in the release > > > > > > > > - Do we need the .api package? Couldn't these types be directly in > > > > o.a.clerezza? After all Clerezza is mainly an API > > > > > > > So, what's the name of this artifact in the group org.apache.clerezza? > > > > > > - I don't like the name .api.impl - Without the ".api" it would be a > > > bit > > > > better, still ".impl" is just very unspecific. > > > > > > > > > > If we know the name of the artifact in the previous question, eg > > > foo, we can call it foo.impl. > > > Or do I misunderstand something here? > > > > > > - It's great the the method in GraphWriter to set the Serializer is > > > now > > > > public. Now I can have the code: > > > > GraphWriter graphWriter = new GraphWriter(); > > > > graphWriter.setSerializer(Serializer.getInstance()); > > > > Before I needed to make a subclass to access the protected methods. > > > > However it seems that graphWriter could access the default > > > > serializer using .getInstance itself, if none is set. > > > > > > > > > > I think current master branch already has public method > setSerializer(). > > > > > > Cheers > > > Hasan > > > > > > > > > > Cheers, > > > > Reto > > > > > > > > > > > > -----Original Message----- > > > > From: Reto Gmür <[email protected]> > > > > Sent: Thursday, March 7, 2019 2:36 PM > > > > To: [email protected] > > > > Subject: RE: [DISCUSS] Release the reunited branch as the master > > > > branch version 8 > > > > > > > > Hi Hasan > > > > > > > > Thanks a lot for the overview. > > > > > > > > While I think it shouldn't be a general requirement to release > > > > everything together, in this case it looks like it would make > > > > things easier to use version 2.0.0 in all modules. WDYT? > > > > > > > > Cheers, > > > > Reto > > > > > > > > -----Original Message----- > > > > From: Hasan <[email protected]> > > > > Sent: Wednesday, March 6, 2019 9:54 PM > > > > To: [email protected] > > > > Subject: Re: [DISCUSS] Release the reunited branch as the master > > > > branch version 8 > > > > > > > > Dear all > > > > > > > > I had a discussion with Reto regarding the versioning of the modules. > > > > Currently, all modules have version 8 after the refactoring. Reto > > > > suggested to keep previous version and increase it according to > > > > semantic versioning. > > > > So, since all modules have a breaking change, I will increase the > > > > major number. > > > > However, some modules are new or the result of a renaming. > > > > > > > > We have these modules in the reunited branch: > > > > > > > > * api (was org.apache.clerezza.commons-rdf:commons-rdf-api > > > > 0.3-SNAPSHOT) > > > > * api.impl (was > > > > org.apache.clerezza.commons-rdf:commons-rdf-impl-utils > > > > 0.3-SNAPSHOT) > > > > * ontologies (was org.apache.clerezza:rdf.ontologies > > > > 1.0.1-SNAPSHOT) > > > > * sparql (was sparql package in org.apache.clerezza:rdf.core > > > > 1.0.2-SNAPSHOT) > > > > * representation (was serializedform package in > > > > org.apache.clerezza:rdf.core 1.0.2-SNAPSHOT) > > > > * test.utils (was org.apache.clerezza:rdf.core.test > > > > 1.0.1-SNAPSHOT) > > > > * dataset (was access package in org.apache.clerezza:rdf.core > > > > 1.0.2-SNAPSHOT) > > > > * api.utils (is a merge of org.apache.clerezza:rdf.utils > > > > 1.0.1-SNAPSHOT with org.apache.clerezza:rdf.scala.utils > > > > 1.0.1-SNAPSHOT) > > > > * jaxrs.rdf.providers (was org.apache.clerezza:jaxrs.rdf.providers > > > > 1.0.1-SNAPSHOT) > > > > > > > > api will have version 1.0.0 > > > > api.impl will have version 1.0.0 > > > > ontologies will have version 2.0.0 sparql will have version 2.0.0 > > > > representation will have version 2.0.0 test.utils will have > > > > version > > > > 2.0.0 dataset will have version 2.0.0 api.utils will have version > > > > 2.0.0 jaxrs.rdf.providers will have version 2.0.0 > > > > > > > > What do you think? > > > > Any objections? > > > > > > > > Kind regards > > > > Hasan > > > > > > > > On Sat, Feb 16, 2019 at 11:12 AM Hasan <[email protected]> wrote: > > > > > > > > > Dear all > > > > > > > > > > The reunited branch of Clerezza ( > > > > > https://gitbox.apache.org/repos/asf/clerezza.git) brings back > > > > > the Clerezza common-rdf ( > > > > > https://gitbox.apache.org/repos/asf/clerezza-rdf-core.git) > > > > > into Clerezza (https://gitbox.apache.org/repos/asf/clerezza.git). > > > > > > > > > > At the same time we refactor Clerezza to > > > > > - remove a cyclic dependency between sparql and access package, > > > > > - have all core functionalities in the top level modules instead > > > > > of under a single rdf module, > > > > > - rename some modules and packages to reflect better their > > > functionality. > > > > > > > > > > The refactoring task is more or less complete. > > > > > > > > > > I think we should release the reunited branch in the near future > > > > > as version 8 of master. > > > > > > > > > > What do you think? > > > > > > > > > > Kind regards > > > > > Hasan > > > > > > > > > > > > > > > > > > > >
