Hi,
while trying to integrate the fix for the bug below i
stumbled upon a new issue:
Caused by: java.lang.NoSuchMethodError:
org.eclipse.ecf.remoteservice.AbstractRemoteService.isReturnAsync(Ljava/lang/Object;Ljava/lang/reflect/Method;[Ljava/lang/Object;)Z
at
org.eclipse.ecf.remoteservice.AbstractRemoteService.invoke(AbstractRemoteService.java:503)
... 5 common frames omitted
I updated the ECF implementation to the latest stable
version 3.14.1. The above problem happens in bundle
org.eclipse.ecf.remoteservice. The class
AbstractRemoteService extends
AbstractAsyncProxyRemoteService which is imported via
package import :
org.eclipse.ecf.remoteservice.asyncproxy;version="[1.0.0,3.0.0)"
I decided to use the bundle
org.eclipse.ecf.remoteservice.asyncproxy version
1.1.0.v20180311-0310 (and not version 2.1. 0 because of
other dependencies) which perfectly fits into that version
range.
The problem is that in that bundle the class
AbstractAsyncProxyRemoteService does not have a method
isReturnAsync which leads to the above error.
Should i file a bug or is that a known problem?
Thanks, bye Peter
Am 19.09.2018 um 10:28 schrieb
Peter Hermsdorf:
Done: https://bugs.eclipse.org/bugs/show_bug.cgi?id=539218
Thanks for looking at the issue!
Bye Peter
Am 18.09.2018 um 19:19 schrieb
Scott Lewis:
Hi
Peter,
I believe this is due to the (incorrect) line in
ImportRegistration.close():
public void close() {
boolean publish = false;
ID containerID = null;
Throwable exception = null;
EndpointDescription endpointDescription =
null;
ImportReference imRef = null;
synchronized (this) {
// only do this once
if (!closed) {
containerID = getContainerID();
exception = getException();
endpointDescription =
getEndpointDescription();
imRef = this.importReference;
publish = importReference.close(this);
this.importReference =
null; <-- this line should
be deleted
closed = true;
}
}
...more code
I believe the importReference should never be set to null
in the ImportRegistration and that it is incorrect that it
did so. Also synchronization needs to be added to
ImportRegistration accessors so that the synchronized
block above prevents access to importReference before
closed = true (which is resulting in NPE I think).
Peter if you would please open a bug, and put your info in
comments I will address in latest version as soon as I
can.
Scott
On 9/18/2018 7:33 AM, Peter Hermsdorf wrote:
Hi,
we sometimes faces the following error:
java.lang.NullPointerException
at
org.eclipse.ecf.osgi.services.remoteserviceadmin.RemoteServiceAdmin$ImportRegistration.getException(RemoteServiceAdmin.java:1139)
at
org.eclipse.ecf.osgi.services.remoteserviceadmin.RemoteServiceAdmin$ImportRegistration.getImportReference(RemoteServiceAdmin.java:1104)
at
org.eclipse.ecf.osgi.services.remoteserviceadmin.RemoteServiceAdmin.getImportedEndpoints(RemoteServiceAdmin.java:519)
The corresponding code in the class ImportRegistration
is
public Throwable getException() {
return
(closed)?updateException:importReference.getException();
}
So obviously importReference is null but closed is
false.
Any ideas what could cause this state? I've seen that
there are changes on this class but wasn't able to
determine if these have something to do with this
handling.
We are still on ECF version 3.13.8 and we are using the
ecf-genric provider.
The above code and the call to getImportedEndpoints is
used to update the connection state in an RCP
application and is asynchronously executed and triggered
by a new event to an RemoteServiceAdminListener .... so
more ore less on service state change.
Any hints are welcome!
Thanks, bye Peter