You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@vxquery.apache.org by Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk> on 2016/07/17 13:35:43 UTC

Issues VXQUERY-210 and VXQUERY-222

Hi,

I have attempted to fix the issues related to JSON serialization[1]. Please
suggest if there's any other types that need not be quoted in json value.

[1] https://github.com/apache/vxquery/pull/101


Thank you.

Yours sincerely,
Riyafa
-- 
Riyafa Abdul Hameed
Undergraduate, University of Moratuwa

Email: riyafa.12@cse.mrt.ac.lk
Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
<http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
<http://twitter.com/Riyafa1>

Re: Issues VXQUERY-210 and VXQUERY-222

Posted by Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>.
Hi,

I have squashed the commits to prepare for merging[1].

[1] https://github.com/apache/vxquery/pull/142

Thank you.

Yours sincerely,
Riyafa

On 2 August 2016 at 11:27, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
wrote:

> Hi,
>
> I have made a new PR[1] with the suggested changes.
>
> Thank you.
>
> Yours sincerely,
> Riyafa
>
> On 19 July 2016 at 14:08, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
> wrote:
>
>> Hi,
>>
>>
>> I have come across an issue when serializing if an XML node is found as a
>> value in an object: According the specification[1], it should be serialized
>> "according to the xml output method and the corresponding serialization
>> parameters ". This does not mention that the resulting value should be
>> quoted. I came across the following object:
>>
>> {
>>     "attribute" : <a attr="content"/>
>> }
>>
>> If this is serialized with quotes for the value:
>>
>> {
>>     "attribute":"<a attr="content"/>"
>> }
>>
>> which is invalid json.
>>
>> How should such a case be handled?
>>
>> [1] http://jsoniq.org/docs/JSONiqExtensionToXQuery/html-
>> single/index.html#idm139680637468832
>>
>> Thank you.
>>
>> Yours sincerely,
>> Riyafa
>>
>> On 19 July 2016 at 13:50, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
>> wrote:
>>
>>> Hi,
>>>
>>> I have extended the testing to all data types[1]. As for the comments I
>>> am not sure I understand what needs to be alphabetized and I don't see any
>>> used for a Switch statement. Please kindly help me understand.
>>>
>>> [1] https://github.com/apache/vxquery/pull/101
>>>
>>> Thank you.
>>>
>>> Yours sincerely,
>>> Riyafa
>>>
>>> On 18 July 2016 at 00:09, Preston Carman <pr...@apache.org> wrote:
>>>
>>>> I posted a few comments. Please extend the testing for all data types.
>>>>
>>>> Thanks
>>>>
>>>> On Sun, Jul 17, 2016 at 6:35 AM, Riyafa Abdul Hameed <
>>>> riyafa.12@cse.mrt.ac.lk> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> I have attempted to fix the issues related to JSON serialization[1].
>>>>> Please suggest if there's any other types that need not be quoted in json
>>>>> value.
>>>>>
>>>>> [1] https://github.com/apache/vxquery/pull/101
>>>>>
>>>>>
>>>>> Thank you.
>>>>>
>>>>> Yours sincerely,
>>>>> Riyafa
>>>>> --
>>>>> Riyafa Abdul Hameed
>>>>> Undergraduate, University of Moratuwa
>>>>>
>>>>> Email: riyafa.12@cse.mrt.ac.lk
>>>>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>>>>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>>>>> <http://twitter.com/Riyafa1>
>>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>> Riyafa Abdul Hameed
>>> Undergraduate, University of Moratuwa
>>>
>>> Email: riyafa.12@cse.mrt.ac.lk
>>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>>> <http://twitter.com/Riyafa1>
>>>
>>
>>
>>
>> --
>> Riyafa Abdul Hameed
>> Undergraduate, University of Moratuwa
>>
>> Email: riyafa.12@cse.mrt.ac.lk
>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>> <http://twitter.com/Riyafa1>
>>
>
>
>
> --
> Riyafa Abdul Hameed
> Undergraduate, University of Moratuwa
>
> Email: riyafa.12@cse.mrt.ac.lk
> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
> <http://twitter.com/Riyafa1>
>



-- 
Riyafa Abdul Hameed
Undergraduate, University of Moratuwa

Email: riyafa.12@cse.mrt.ac.lk
Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
<http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
<http://twitter.com/Riyafa1>

Re: Issues VXQUERY-210 and VXQUERY-222

Posted by Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>.
Hi,

I have made a new PR[1] with the suggested changes.

Thank you.

Yours sincerely,
Riyafa

On 19 July 2016 at 14:08, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
wrote:

> Hi,
>
>
> I have come across an issue when serializing if an XML node is found as a
> value in an object: According the specification[1], it should be serialized
> "according to the xml output method and the corresponding serialization
> parameters ". This does not mention that the resulting value should be
> quoted. I came across the following object:
>
> {
>     "attribute" : <a attr="content"/>
> }
>
> If this is serialized with quotes for the value:
>
> {
>     "attribute":"<a attr="content"/>"
> }
>
> which is invalid json.
>
> How should such a case be handled?
>
> [1]
> http://jsoniq.org/docs/JSONiqExtensionToXQuery/html-single/index.html#idm139680637468832
>
> Thank you.
>
> Yours sincerely,
> Riyafa
>
> On 19 July 2016 at 13:50, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
> wrote:
>
>> Hi,
>>
>> I have extended the testing to all data types[1]. As for the comments I
>> am not sure I understand what needs to be alphabetized and I don't see any
>> used for a Switch statement. Please kindly help me understand.
>>
>> [1] https://github.com/apache/vxquery/pull/101
>>
>> Thank you.
>>
>> Yours sincerely,
>> Riyafa
>>
>> On 18 July 2016 at 00:09, Preston Carman <pr...@apache.org> wrote:
>>
>>> I posted a few comments. Please extend the testing for all data types.
>>>
>>> Thanks
>>>
>>> On Sun, Jul 17, 2016 at 6:35 AM, Riyafa Abdul Hameed <
>>> riyafa.12@cse.mrt.ac.lk> wrote:
>>>
>>>> Hi,
>>>>
>>>> I have attempted to fix the issues related to JSON serialization[1].
>>>> Please suggest if there's any other types that need not be quoted in json
>>>> value.
>>>>
>>>> [1] https://github.com/apache/vxquery/pull/101
>>>>
>>>>
>>>> Thank you.
>>>>
>>>> Yours sincerely,
>>>> Riyafa
>>>> --
>>>> Riyafa Abdul Hameed
>>>> Undergraduate, University of Moratuwa
>>>>
>>>> Email: riyafa.12@cse.mrt.ac.lk
>>>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>>>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>>>> <http://twitter.com/Riyafa1>
>>>>
>>>
>>>
>>
>>
>> --
>> Riyafa Abdul Hameed
>> Undergraduate, University of Moratuwa
>>
>> Email: riyafa.12@cse.mrt.ac.lk
>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>> <http://twitter.com/Riyafa1>
>>
>
>
>
> --
> Riyafa Abdul Hameed
> Undergraduate, University of Moratuwa
>
> Email: riyafa.12@cse.mrt.ac.lk
> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
> <http://twitter.com/Riyafa1>
>



-- 
Riyafa Abdul Hameed
Undergraduate, University of Moratuwa

Email: riyafa.12@cse.mrt.ac.lk
Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
<http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
<http://twitter.com/Riyafa1>

Re: Issues VXQUERY-210 and VXQUERY-222

Posted by Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>.
Hi,


I have come across an issue when serializing if an XML node is found as a
value in an object: According the specification[1], it should be serialized
"according to the xml output method and the corresponding serialization
parameters ". This does not mention that the resulting value should be
quoted. I came across the following object:

{
    "attribute" : <a attr="content"/>
}

If this is serialized with quotes for the value:

{
    "attribute":"<a attr="content"/>"
}

which is invalid json.

How should such a case be handled?

[1]
http://jsoniq.org/docs/JSONiqExtensionToXQuery/html-single/index.html#idm139680637468832

Thank you.

Yours sincerely,
Riyafa

On 19 July 2016 at 13:50, Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>
wrote:

> Hi,
>
> I have extended the testing to all data types[1]. As for the comments I am
> not sure I understand what needs to be alphabetized and I don't see any
> used for a Switch statement. Please kindly help me understand.
>
> [1] https://github.com/apache/vxquery/pull/101
>
> Thank you.
>
> Yours sincerely,
> Riyafa
>
> On 18 July 2016 at 00:09, Preston Carman <pr...@apache.org> wrote:
>
>> I posted a few comments. Please extend the testing for all data types.
>>
>> Thanks
>>
>> On Sun, Jul 17, 2016 at 6:35 AM, Riyafa Abdul Hameed <
>> riyafa.12@cse.mrt.ac.lk> wrote:
>>
>>> Hi,
>>>
>>> I have attempted to fix the issues related to JSON serialization[1].
>>> Please suggest if there's any other types that need not be quoted in json
>>> value.
>>>
>>> [1] https://github.com/apache/vxquery/pull/101
>>>
>>>
>>> Thank you.
>>>
>>> Yours sincerely,
>>> Riyafa
>>> --
>>> Riyafa Abdul Hameed
>>> Undergraduate, University of Moratuwa
>>>
>>> Email: riyafa.12@cse.mrt.ac.lk
>>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>>> <http://twitter.com/Riyafa1>
>>>
>>
>>
>
>
> --
> Riyafa Abdul Hameed
> Undergraduate, University of Moratuwa
>
> Email: riyafa.12@cse.mrt.ac.lk
> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
> <http://twitter.com/Riyafa1>
>



-- 
Riyafa Abdul Hameed
Undergraduate, University of Moratuwa

Email: riyafa.12@cse.mrt.ac.lk
Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
<http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
<http://twitter.com/Riyafa1>

Re: Issues VXQUERY-210 and VXQUERY-222

Posted by Riyafa Abdul Hameed <ri...@cse.mrt.ac.lk>.
Hi,

I have extended the testing to all data types[1]. As for the comments I am
not sure I understand what needs to be alphabetized and I don't see any
used for a Switch statement. Please kindly help me understand.

[1] https://github.com/apache/vxquery/pull/101

Thank you.

Yours sincerely,
Riyafa

On 18 July 2016 at 00:09, Preston Carman <pr...@apache.org> wrote:

> I posted a few comments. Please extend the testing for all data types.
>
> Thanks
>
> On Sun, Jul 17, 2016 at 6:35 AM, Riyafa Abdul Hameed <
> riyafa.12@cse.mrt.ac.lk> wrote:
>
>> Hi,
>>
>> I have attempted to fix the issues related to JSON serialization[1].
>> Please suggest if there's any other types that need not be quoted in json
>> value.
>>
>> [1] https://github.com/apache/vxquery/pull/101
>>
>>
>> Thank you.
>>
>> Yours sincerely,
>> Riyafa
>> --
>> Riyafa Abdul Hameed
>> Undergraduate, University of Moratuwa
>>
>> Email: riyafa.12@cse.mrt.ac.lk
>> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
>> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
>> <http://twitter.com/Riyafa1>
>>
>
>


-- 
Riyafa Abdul Hameed
Undergraduate, University of Moratuwa

Email: riyafa.12@cse.mrt.ac.lk
Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
<http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
<http://twitter.com/Riyafa1>

Re: Issues VXQUERY-210 and VXQUERY-222

Posted by Preston Carman <pr...@apache.org>.
I posted a few comments. Please extend the testing for all data types.

Thanks

On Sun, Jul 17, 2016 at 6:35 AM, Riyafa Abdul Hameed <
riyafa.12@cse.mrt.ac.lk> wrote:

> Hi,
>
> I have attempted to fix the issues related to JSON serialization[1].
> Please suggest if there's any other types that need not be quoted in json
> value.
>
> [1] https://github.com/apache/vxquery/pull/101
>
>
> Thank you.
>
> Yours sincerely,
> Riyafa
> --
> Riyafa Abdul Hameed
> Undergraduate, University of Moratuwa
>
> Email: riyafa.12@cse.mrt.ac.lk
> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/>
> <http://facebook.com/riyafa.ahf>  <http://lk.linkedin.com/in/riyafa>
> <http://twitter.com/Riyafa1>
>