You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by Ciprian Ciubotariu <ch...@gmx.net> on 2015/07/18 18:45:09 UTC
Renaming packages to org.apache.brooklyn
As part of the apache process, brooklyn needs to rename its packages with the
org.apache prefix. I have already started the process and refactored some
projects.
My branch for this effort is https://github.com/CMoH/incubator-brooklyn/tree/rename_org_apache, based on the master 0.8.0 branch, from which
I will regularly create PRs.
As I am approaching the core packages, the core API is bound to change more
and more. I hope you will support my efforts by attempting to process the
current PR queue, reviewing my PRs in this regard and maybe adding your own
efforts to this refactoring.
I suggest that the the first 0.8.x releases be used to weed out issues related
to the rename, while allowing the brooklyn community (downstream projects,
etc) to catch up. (They will have to adapt to the new naming scheme when 0.8.0
gets released)
So far, the most sensitive issues I encountered are:
1. YAML blueprints that contain classnames
2. migration of persisted names
3. REST endpoints that change their URLs
For #1, we can either change the YAML files to add the org.apache prefix, or
choose a naming scheme that does not publicize the class package name (like
URNs, or even the old naming scheme).
For #3, I would choose to update the REST service consumers to the new
resource paths (i.e. /v1/apidoc/brooklyn.rest.resources.ApidocResource becomes
/v1/apidoc/org.apache.brooklyn.rest.resources.ApidocResource) and have the
clients adjust their code for 0.8.0.
I await your comments and advice on how to handle these issues (and maybe
other problems that I have not yet encountered).
Ciprian
Re: Renaming packages to org.apache.brooklyn
Posted by Ciprian Ciubotariu <ch...@gmx.net>.
So far, I have found that the Apidoc REST links have changed, but the rest of
the URLs remained the same. Maybe issue #3 is not a real problem, and I was
mislead by a test case.
Updated PR #755. All tests pass.
On Saturday 18 July 2015 19:45:09 Ciprian Ciubotariu wrote:
> As part of the apache process, brooklyn needs to rename its packages with
> the org.apache prefix. I have already started the process and refactored
> some projects.
>
> My branch for this effort is
> https://github.com/CMoH/incubator-brooklyn/tree/rename_org_apache, based on
> the master 0.8.0 branch, from which I will regularly create PRs.
>
> As I am approaching the core packages, the core API is bound to change more
> and more. I hope you will support my efforts by attempting to process the
> current PR queue, reviewing my PRs in this regard and maybe adding your own
> efforts to this refactoring.
>
>
> I suggest that the the first 0.8.x releases be used to weed out issues
> related to the rename, while allowing the brooklyn community (downstream
> projects, etc) to catch up. (They will have to adapt to the new naming
> scheme when 0.8.0 gets released)
>
> So far, the most sensitive issues I encountered are:
>
> 1. YAML blueprints that contain classnames
> 2. migration of persisted names
> 3. REST endpoints that change their URLs
>
>
> For #1, we can either change the YAML files to add the org.apache prefix, or
> choose a naming scheme that does not publicize the class package name (like
> URNs, or even the old naming scheme).
>
> For #3, I would choose to update the REST service consumers to the new
> resource paths (i.e. /v1/apidoc/brooklyn.rest.resources.ApidocResource
> becomes /v1/apidoc/org.apache.brooklyn.rest.resources.ApidocResource) and
> have the clients adjust their code for 0.8.0.
>
>
> I await your comments and advice on how to handle these issues (and maybe
> other problems that I have not yet encountered).
>
> Ciprian