-
1. Re: backport-concurrent-util
adrian.brock Feb 5, 2006 10:04 AM (in response to adrian.brock)cvs is not working, so I've downloaded the tarball.
I'm going to look at it. -
2. Re: backport-concurrent-util
adrian.brock Feb 5, 2006 1:41 PM (in response to adrian.brock)I've added the tests.
The main problem I can see is that some use a timeout of 300ms
which means they can fail randomly when the host is busy doing something else.
This is configurable via the system property "tck.shortDelay".
There are also some stress tests in test/loops, but they aren't junit tests. -
3. Re: backport-concurrent-util
starksm64 Feb 5, 2006 3:50 PM (in response to adrian.brock)I see a new concurrent cvs module for this. This only compiles under jdk1.4.2 due to conflicts with java.util.* imports that look to be caused by the wildard import usage. Would't hurt to require building with 1.4.x would it?
-
4. Re: backport-concurrent-util
adrian.brock Feb 6, 2006 4:37 AM (in response to adrian.brock)Done.
-
5. Re: backport-concurrent-util
adrian.brock Feb 6, 2006 5:07 AM (in response to adrian.brock)I've implemented a retrocheck task, sample output:
[ejort@htimes2 jbossretro]$ ant tests Buildfile: build.xml check: [retrocheck] Check Successful: 1014 ms [retrocheck] ==== /home/ejort/retro/workspace/jbossretro/output/test-classes14/org/jboss/test/enums/SimpleEnumTypesTestCase.class [retrocheck] Cannot find method org.jboss.util.EnumSet.range(Lorg/jboss/lang/Enum;Lorg/jboss/lang/Enum;)Lorg/jboss/util/EnumSet; [retrocheck] Cannot find method org.jboss.util.EnumSet.allOf(Ljava/lang/Class;)Lorg/jboss/util/EnumSet; [retrocheck] Cannot find method org.jboss.util.EnumSet.complementOf(Lorg/jboss/util/EnumSet;)Lorg/jboss/util/EnumSet; [retrocheck] ==== /home/ejort/retro/workspace/jbossretro/output/test-classes14/org/jboss/test/util/AllConcurrentTestCase.class [retrocheck] Cannot find method edu.emory.mathcs.backport.java.util.concurrent.DelayQueue.add(Ledu/emory/mathcs/backport/java/util/concurrent/Delayed;)Z [retrocheck] ==== /home/ejort/retro/workspace/jbossretro/output/test-classes14/org/jboss/test/annotations/AnnotationsTestCase.class [retrocheck] Cannot find method java.lang.reflect.Method.isAnnotationPresent(Ljava/lang/Class;)Z [retrocheck] Cannot find method java.lang.reflect.Method.getAnnotation(Ljava/lang/Class;)Lorg/jboss/lang/Annotation; [retrocheck] Cannot find method java.lang.reflect.Method.getAnnotations()[Lorg/jboss/lang/Annotation; [retrocheck] ==== /home/ejort/retro/workspace/jbossretro/output/test-classes14/org/jboss/test/enums/Card.class [retrocheck] Cannot find constructor org.jboss.util.EnumMap(Ljava/lang/Class;)V [retrocheck] Cannot find constructor org.jboss.util.EnumMap(Ljava/lang/Class;)V BUILD FAILED /home/ejort/retro/workspace/jbossretro/build.xml:180: Java returned: 1
-
6. Re: backport-concurrent-util
ryan.campbell Feb 6, 2006 10:50 PM (in response to adrian.brock)I guess we want these tests under cruisecontrol if we are going to be maintaining our own fork?
I'm not sure what extra coverage we could provide besides using the AS testsuite (which would require making the big leap) or some detailed analysis of the existing test coverage to see what tests need writing. -
7. Re: backport-concurrent-util
adrian.brock Feb 7, 2006 5:23 AM (in response to adrian.brock)I don't want to fork the project unless we have to.
And even then only temporarily.
I put it in CVS so we have a consistent build (the base relase has no ant scripts)
with our own tags and releases.
Fixes should be contributed back to backport-concurrent-util
Yes. Please add the tests to cruisecontrol.
We also need to investigate the loop tests. The doco says these
are also good tests of whether the OS/JVM supports SMP properly.
So we might want to use them as a basis of a generic tool available to users
to validate their environment! -
8. Re: backport-concurrent-util
ryan.campbell Feb 8, 2006 5:21 PM (in response to adrian.brock)Testsuite automation task: http://jira.jboss.com/jira/browse/JBQA-291
Just to refresh my memory, the idea here is to use the retroweaver to map java5's java.util.concurrent to the equivalent backport packages. So this jar will need to be on the classpath of any retroweaved classes. -
9. Re: backport-concurrent-util
adrian.brock Feb 9, 2006 5:01 AM (in response to adrian.brock)That is correct.