You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Tobias Bocanegra (JIRA)" <ji...@apache.org> on 2008/01/25 03:16:34 UTC

[jira] Created: (SLING-194) use default format for JSON resource loader

use default format for JSON resource loader 
--------------------------------------------

                 Key: SLING-194
                 URL: https://issues.apache.org/jira/browse/SLING-194
             Project: Sling
          Issue Type: Improvement
          Components: Repository
            Reporter: Tobias Bocanegra


the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.

suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (SLING-194) use default format for JSON resource loader

Posted by "Tobias Bocanegra (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tobias Bocanegra closed SLING-194.
----------------------------------


thanks.

> use default format for JSON resource loader 
> --------------------------------------------
>
>                 Key: SLING-194
>                 URL: https://issues.apache.org/jira/browse/SLING-194
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repository
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>             Fix For: 2.0.0
>
>         Attachments: loader-xjson.r616687.patch
>
>
> the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SLING-194) use default format for JSON resource loader

Posted by "Tobias Bocanegra (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tobias Bocanegra updated SLING-194:
-----------------------------------

    Attachment: loader-xjson.r616687.patch

providing patch for Loader that implements content loading from default 
json serialized content. currently uses special extension: .xjson

> use default format for JSON resource loader 
> --------------------------------------------
>
>                 Key: SLING-194
>                 URL: https://issues.apache.org/jira/browse/SLING-194
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repository
>            Reporter: Tobias Bocanegra
>         Attachments: loader-xjson.r616687.patch
>
>
> the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (SLING-194) use default format for JSON resource loader

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger reassigned SLING-194:
---------------------------------------

    Assignee: Felix Meschberger

> use default format for JSON resource loader 
> --------------------------------------------
>
>                 Key: SLING-194
>                 URL: https://issues.apache.org/jira/browse/SLING-194
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repository
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>         Attachments: loader-xjson.r616687.patch
>
>
> the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SLING-194) use default format for JSON resource loader

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-194.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 2.0.0

Patch is applied. Please close if this issue is solved for you.

Replacing the standard JSON import by the new one provided here will make it into a new issue.

> use default format for JSON resource loader 
> --------------------------------------------
>
>                 Key: SLING-194
>                 URL: https://issues.apache.org/jira/browse/SLING-194
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repository
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>             Fix For: 2.0.0
>
>         Attachments: loader-xjson.r616687.patch
>
>
> the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Commented: (SLING-194) use default format for JSON resource loader

Posted by Felix Meschberger <fm...@gmail.com>.
Hi all,

I would like to put on discussion, whether we drop the former very
simple format of JSON in favor of the format generated by the
JsonWriter. This would allow for simple round-tripping and would also
not generate to many different and incompatible formats.

The result of this would be to replace JsonReader by the current
XJsonReader and the requirement to modify any existing .json files used
for initial content loading. And of course the temporary .xjson
extension would not be supported any more.

WDYT ?

Regards
Felix

Am Montag, den 04.02.2008, 12:17 -0800 schrieb Felix Meschberger (JIRA):
> [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12565502#action_12565502 ] 
> 
> Felix Meschberger commented on SLING-194:
> -----------------------------------------
> 
> Applied modified patch in Rev 618414.
> 
> The patch is modified such, that XJsonReader extends JsonReader and overwrites createNode and createResource methods.
> 
> I think, we may well drop the old JSON format support implemented by JsonReader in favor of the "standard" JsonWriter format.
> 
> > use default format for JSON resource loader 
> > --------------------------------------------
> >
> >                 Key: SLING-194
> >                 URL: https://issues.apache.org/jira/browse/SLING-194
> >             Project: Sling
> >          Issue Type: Improvement
> >          Components: Repository
> >            Reporter: Tobias Bocanegra
> >            Assignee: Felix Meschberger
> >         Attachments: loader-xjson.r616687.patch
> >
> >
> > the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> > suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.
> 


[jira] Commented: (SLING-194) use default format for JSON resource loader

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12565502#action_12565502 ] 

Felix Meschberger commented on SLING-194:
-----------------------------------------

Applied modified patch in Rev 618414.

The patch is modified such, that XJsonReader extends JsonReader and overwrites createNode and createResource methods.

I think, we may well drop the old JSON format support implemented by JsonReader in favor of the "standard" JsonWriter format.

> use default format for JSON resource loader 
> --------------------------------------------
>
>                 Key: SLING-194
>                 URL: https://issues.apache.org/jira/browse/SLING-194
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repository
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>         Attachments: loader-xjson.r616687.patch
>
>
> the current org.apache.sling.jcr.resource.internal.loader.JsonReader has a pretty simple format for loading content from the bundles into the repository. the pain is that if you already have repository content, you can't create just a JSON dump and put it in the resource.
> suggest to change the JsonReader to accept a JSON dump as provided by a JSON content export by the default servlet.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.