On 8/21/2010 10:17 PM, Patricia Shanahan wrote:
...
Tomorrow, unless I get a better idea or someone posts one, I'll start a
binary search. The objective will be to find consecutive revisions N and
N+1 such that N passes the servicediscovery tests and N+1 fails them.
...
I have preliminary results from the binary search. I can't narrow it
down to a single check-in because I cannot build revision 934802. The
indications are that revision 934258 is the last buildable revision that
passes and revision 935130 is the first buildable revision that fails.
This is based on a single test,
com/sun/jini/test/impl/servicediscovery/event/NotifyEventDropProxyTaskRace.td,
that I had previously found to be solidly failing on repeated runs on
the latest revision.
I'm in the process of running the full QA tests, servicediscovery
included, to see if the other failing tests behave the same way.
Patricia
Revision: 935130
Author: peter_firmstone
Date: 10:12:18 PM, Friday, April 16, 2010
Message:
New signatures containing MarshalledInstance were too similar to those
containing MarshalledObject, such that when a null parameter was given
by client code, the correct method signature to select was ambiguous and
caused a compile time error.
New signatures changed slightly to avoid ambiguous matches.
----
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/StreamingServiceRegistrar.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/event/RemoteEvent.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/ServiceEvent.java
Revision: 934802
Author: peter_firmstone
Date: 4:54:22 AM, Friday, April 16, 2010
Message:
Experimental commit, please feel free to read / participate / discuss.
----
Modified :
/incubator/river/jtsk/trunk/src/net/jini/io/CDCMarshalledObject.java
Modified : /incubator/river/jtsk/trunk/src/net/jini/io/Convert.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/io/FromMOInputStream.java
Modified : /incubator/river/jtsk/trunk/src/net/jini/io/MarshalledObject.java
Modified : /incubator/river/jtsk/trunk/src/net/jini/io/ToMOInputStream.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/CodebaseAccessClassLoader.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/CodebaseAccessOverridePermission.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/event/RemoteEvent.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/Converter.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/ToMOOutputStream.java
Added :
/incubator/river/jtsk/trunk/test/src/net/jini/io/ToMOOutputStreamTest.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/ServiceEvent.java
Revision: 934258
Author: peter_firmstone
Date: 6:11:12 PM, Wednesday, April 14, 2010
Message:
This patch integrates changes to replace the RMIClassLoaderSPI with a
CodebaseAccessClassLoader as per River-336. Although it doesn't include
all changes to PreferredClassProvider.
Additional changes to net.jini.io have been made to allow a subset of
this package to be built at a later stage for CDC Personal Profile 1.11
(a subset of Java 1.4.2).
An additional interface has been created, which is a subset of
ServiceRegistrar, which ServiceRegistrar now extends, this is to enable
building a release for CDC Personal Profile 1.11 or greater at a later
point in time, without causing any breakage to existing application
software.
Furthermore additional permissions have been granted to the qa test
defaulttest.policy in order for some random test failures to pass. This
was specific to the Java 1.6 platform and I believe related to the
-source=5 and -target=jsr14 compiler optimisations that load a security
manager earlier, it is also related to the removal of the need to
utilise the jre/etc directory for jsk-policy.jar
These changes are experimental, and some incomplete, please feel free to
make comments and reccommendations or hightlite any unforseen issues.
The Apache River version has been changed to 2.2.0
----
Modified : /incubator/river/jtsk/trunk
Modified : /incubator/river/jtsk/trunk/common.xml
Modified : /incubator/river/jtsk/trunk/qa/build.xml
Modified : /incubator/river/jtsk/trunk/qa/harness/policy/defaulttest.policy
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/constants/VersionConstants.java
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/norm/NormServerBaseImpl.java
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/outrigger/EntryRep.java
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/phoenix/ActivationGroupImpl.java
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/phoenix/ActivationGroupInit.java
Modified :
/incubator/river/jtsk/trunk/src/com/sun/jini/reggie/EntryClassBase.java
Modified : /incubator/river/jtsk/trunk/src/manifest/jsk-platform.mf
Modified :
/incubator/river/jtsk/trunk/src/net/jini/activation/ActivationGroup.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/PortableServiceRegistrar.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/ServiceRegistrar.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/core/lookup/StreamingServiceRegistrar.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/CDCMarshalledObject.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/Convert.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/FromMOInputStream.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/io/MarshalInputStream.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/io/MarshalOutputStream.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/io/MarshalledInstance.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/io/MarshalledInstanceInputStream.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/io/MarshalledInstanceOutputStream.java
Modified : /incubator/river/jtsk/trunk/src/net/jini/io/MarshalledObject.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/PackageVersion.java
Added : /incubator/river/jtsk/trunk/src/net/jini/io/ToMOInputStream.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/ClassAnnotation.java
Modified : /incubator/river/jtsk/trunk/src/net/jini/loader/ClassLoading.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/loader/CodebaseAccessClassLoader.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/loader/CodebaseAccessOverridePermission.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/loader/CodebaseClassAccess.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/DownloadPermission.java
Added :
/incubator/river/jtsk/trunk/src/net/jini/loader/RMIClassLoaderCodebaseAccess.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/pref/PreferredClassProvider.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/loader/pref/RequireDlPermProvider.java
Modified :
/incubator/river/jtsk/trunk/src/net/jini/security/proxytrust/ProxyTrustVerifier.java
Added : /incubator/river/jtsk/trunk/test/src/net/jini/io
Added : /incubator/river/jtsk/trunk/test/src/net/jini/io/ConvertTest.java
Added :
/incubator/river/jtsk/trunk/test/src/net/jini/io/PackageVersionTest.java
