[dev] Review to a patchset for collection payload

2016-03-03 Thread Maloor, Kishen
>> However, resource spec based filtering looks strange because >> IoTivity cannot follows the whole spec context and update. > > Sorry, I didn't follow this one. Thiago, If I understood both of you correctly, I think Uze is confused by your example in which you mentioned that "com.samsung.freez

[dev] Review to a patchset for collection payload

2016-03-03 Thread 최우제(Uze Choi)
: ???; Habib Virji; iotivity-dev at lists.iotivity.org; 'Jon A. Cruz' Subject: Re: [dev] Review to a patchset for collection payload On quinta-feira, 3 de mar?o de 2016 02:47:19 PST ??? wrote: > Hi. Thiago, > > What you've pointed as a problem sounds less persuasive to me. >

[dev] Review to a patchset for collection payload

2016-03-03 Thread 최우제(Uze Choi)
on A. Cruz [mailto:j...@osg.samsung.com] Sent: Thursday, March 03, 2016 3:22 AM To: jihun.ha at samsung.com; thiago.macieira at intel.com; ???; Habib Virji Cc: iotivity-dev at lists.iotivity.org Subject: Re: [dev] Review to a patchset for collection payload I'll give it another pass now

[dev] Review to a patchset for collection payload

2016-03-03 Thread Thiago Macieira
On quinta-feira, 3 de mar?o de 2016 08:38:12 PST Maloor, Kishen wrote: > Thiago, > > If I understood both of you correctly, I think Uze is confused > by your example in which you mentioned that "com.samsung.freezer? > MAY have a property ?temp? because its already reserved by some OCF > standard r

[dev] Review to a patchset for collection payload

2016-03-03 Thread Thiago Macieira
On quinta-feira, 3 de mar?o de 2016 15:20:32 PST ???(Uze Choi) wrote: > Your Change request is independent from this patch which means your -1 > should be set into new Jira ticket. Fair enough. I will re-review in that light. I will also send a note to the developers of the certification test too

[dev] Review to a patchset for collection payload

2016-03-03 Thread 하지훈
An HTML attachment was scrubbed... URL: -- next part -- A non-text attachment was scrubbed... Name: 201603031147209_AWR5WPSS.gif Type: image/gif Size: 13168 bytes Desc: not avai

[dev] Review to a patchset for collection payload

2016-03-02 Thread Thiago Macieira
On quinta-feira, 3 de mar?o de 2016 02:47:19 PST ??? wrote: > Hi. Thiago, > > What you've pointed as a problem sounds less persuasive to me. > > If one wants to define a vendor-specific attribute, OIC specification > already gives a certain rule for naming: > Resource Attribute :x_[vender]_

[dev] Review to a patchset for collection payload

2016-03-02 Thread Thiago Macieira
On quinta-feira, 3 de mar?o de 2016 10:26:21 PST ???(Uze Choi) wrote: > Hi Thiago & Jon, > > Payload update have been done already on 1.0.1 release to sync with spec. > > Habib(sub-maintainer) submitted most of those patches successfully. > > However, we figure out some bug from the collection r

[dev] Review to a patchset for collection payload

2016-03-02 Thread 하지훈
An HTML attachment was scrubbed... URL: -- next part -- A non-text attachment was scrubbed... Name: 201603022138393_QKNMBDIF.gif Type: image/gif Size: 13168 bytes Desc: not avai

[dev] Review to a patchset for collection payload

2016-03-02 Thread Jon A. Cruz
I'll give it another pass now (did last night and only saw a few minor follow-up items), but the main point is that since Thiago had given it consecutive -1 reviews it would be best to see if it finally corrected all of his concerns. On 03/02/2016 04:38 AM, ??? wrote: > Samsung Enterprise Porta