Eclipse Community Forums: Eclipse Communications Framework (ECF) » Question about method asProperties( ) in ServiceProperties (in ECF discovery)

Home » Archived » Eclipse Communications Framework (ECF) » Question about method asProperties( ) in ServiceProperties (in ECF discovery)

Show: Today's Messages :: Show Polls :: Message Navigator

Question about method asProperties( ) in ServiceProperties (in ECF discovery) [message #623462]

Fri, 25 April 2008 13:50

Phuong Hoang

Messages: 10
Registered: July 2009

Junior Member

Hi again,

I have a question about the behavior of method asProperties() in
ServiceProperties

public Properties asProperties( ){

return new Properties(props);

}

=> This does not return copy of Properties object BUT return an empty
Properties with the default list containing the same elements as variable
ServiceProperties.props instead. Is this what it supposed to behave or is
this a bug???

The reason I ask because using the jslp provider for ECF, when register the
service with non-empty ServiceProperties it results in service registered
with no attribute registered. This happens because when jslp enumerates the
list of attributes to register for the service, it applies method
java.util.Dictionary.keys( ) on ServiceProperties.asProperties( ) and got an
empty list returned (since keys in non-empty default list is not considered
as existing keys, I think).

Phuong

Report message to a moderator

Re: Question about method asProperties( ) in ServiceProperties (in ECF discovery) [message #623464 is a reply to message #623462]

Fri, 25 April 2008 17:44

Scott Lewis

Messages: 1038
Registered: July 2009

Senior Member

Hi Phuong,

Phuong Hoang wrote:
> Hi again,
>
> I have a question about the behavior of method asProperties() in
> ServiceProperties
>
> public Properties asProperties( ){
>
> return new Properties(props);
>
> }
>
> => This does not return copy of Properties object BUT return an empty
> Properties with the default list containing the same elements as variable
> ServiceProperties.props instead. Is this what it supposed to behave or is
> this a bug???

Seems like the specification of IServiceProperties.asProperties() could
go either way to me. The specification (at least so far) is vague on
this point. Any other opinions/views about whether it should be a copy
or not?

>
> The reason I ask because using the jslp provider for ECF, when register the
> service with non-empty ServiceProperties it results in service registered
> with no attribute registered. This happens because when jslp enumerates the
> list of attributes to register for the service, it applies method
> java.util.Dictionary.keys( ) on ServiceProperties.asProperties( ) and got an
> empty list returned (since keys in non-empty default list is not considered
> as existing keys, I think).

I've modified the bug you filed on a related issue to accomodate this:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=228882

The jSLP provider *should* be including service attributes in the
IServiceProperties associated with the IServiceInfo. Currently it is not.

Thanks for the reports/observations.

Scott

Report message to a moderator

Re: Question about method asProperties( ) in ServiceProperties (in ECF discovery) [message #623467 is a reply to message #623464]

Mon, 28 April 2008 15:39

Markus Kuppe

Messages: 177
Registered: July 2009

Senior Member

Scott Lewis wrote:
> Hi Phuong,
>
> Phuong Hoang wrote:
>> Hi again,
>>
>> I have a question about the behavior of method asProperties() in
>> ServiceProperties
>>
>> public Properties asProperties( ){
>>
>> return new Properties(props);
>>
>> }
>>
>> => This does not return copy of Properties object BUT return an empty
>> Properties with the default list containing the same elements as
>> variable ServiceProperties.props instead. Is this what it supposed to
>> behave or is this a bug???
>
>
> Seems like the specification of IServiceProperties.asProperties() could
> go either way to me. The specification (at least so far) is vague on
> this point. Any other opinions/views about whether it should be a copy
> or not?
>
>>
>> The reason I ask because using the jslp provider for ECF, when
>> register the service with non-empty ServiceProperties it results in
>> service registered with no attribute registered. This happens because
>> when jslp enumerates the list of attributes to register for the
>> service, it applies method java.util.Dictionary.keys( ) on
>> ServiceProperties.asProperties( ) and got an empty list returned
>> (since keys in non-empty default list is not considered as existing
>> keys, I think).

What do people think if we remove
org.eclipse.ecf.discovery.IServiceInfo.getServiceProperties( )? Back when
I added the method it seems reasonable. But with the service
attributes/properties overhaul, the method doesn't make sense any more.

Cheers
Markus

Report message to a moderator

Re: Question about method asProperties( ) in ServiceProperties (in ECF discovery) [message #623468 is a reply to message #623467]

Mon, 28 April 2008 15:49

Markus Kuppe

Messages: 177
Registered: July 2009

Senior Member

Markus Alexander Kuppe wrote:
> Scott Lewis wrote:
>> Hi Phuong,
>>
>> Phuong Hoang wrote:
>>> Hi again,
>>>
>>> I have a question about the behavior of method asProperties() in
>>> ServiceProperties
>>>
>>> public Properties asProperties( ){
>>>
>>> return new Properties(props);
>>>
>>> }
>>>
>>> => This does not return copy of Properties object BUT return an empty
>>> Properties with the default list containing the same elements as
>>> variable ServiceProperties.props instead. Is this what it supposed to
>>> behave or is this a bug???
>>
>>
>> Seems like the specification of IServiceProperties.asProperties()
>> could go either way to me. The specification (at least so far) is
>> vague on this point. Any other opinions/views about whether it should
>> be a copy or not?
>>
>>>
>>> The reason I ask because using the jslp provider for ECF, when
>>> register the service with non-empty ServiceProperties it results in
>>> service registered with no attribute registered. This happens because
>>> when jslp enumerates the list of attributes to register for the
>>> service, it applies method java.util.Dictionary.keys( ) on
>>> ServiceProperties.asProperties( ) and got an empty list returned
>>> (since keys in non-empty default list is not considered as existing
>>> keys, I think).
>
> What do people think if we remove
> org.eclipse.ecf.discovery.IServiceInfo.getServiceProperties( )? Back when
> I added the method it seems reasonable. But with the service
> attributes/properties overhaul, the method doesn't make sense any more.

Looking at the test cases, maybe size() and equals()/hashCode() should
be exposed in IServiceProperties.

Markus

Report message to a moderator

Previous Topic:	How to implement a new provider
Next Topic:	Questions about ECF discovery with jSLP provider

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Sat Apr 20 02:33:08 GMT 2024

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter