X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F99rh_perl_perf_bug.t;h=4b275fba048e8adc7ebb2a3028ed72a44417c400;hb=2c96eeed34d7fa42dbb9dbae04a93a9dc0048e2e;hp=645c115294a4d31be36e5c54e0e03d7a52e5f813;hpb=dc253b7737070a66859d615ec79522522a0b1e8e;p=dbsrgits%2FDBIx-Class-Historic.git

diff --git a/t/99rh_perl_perf_bug.t b/t/99rh_perl_perf_bug.t
index 645c115..4b275fb 100644
--- a/t/99rh_perl_perf_bug.t
+++ b/t/99rh_perl_perf_bug.t
@@ -3,7 +3,6 @@ use strict;
 use warnings;
 use Test::More;
 use lib qw(t/lib);
-use DBICTest;
 
 # This is a rather unusual test.
 # It does not test any aspect of DBIx::Class, but instead tests the
@@ -16,57 +15,99 @@ use DBICTest;
 # variable
 #
 # If these tests fail then please read the section titled
-# Perl Performance Issues on Red Hat Systems in 
+# Perl Performance Issues on Red Hat Systems in
 # L<DBIx::Class::Manual::Troubleshooting>
 
 plan skip_all =>
   'Skipping RH perl performance bug tests as DBIC_NO_WARN_BAD_PERL set'
   if ( $ENV{DBIC_NO_WARN_BAD_PERL} );
 
-eval "use Benchmark";
-plan skip_all => 'needs Benchmark for testing' if $@;
-
-plan tests => 2;
+plan skip_all => 'Skipping as AUTOMATED_TESTING is set'
+  if ( $ENV{AUTOMATED_TESTING} );
 
-{
-    package    # don't want this in PAUSE
-      TestRHBug;
-    use overload bool => sub { 0 }
-}
+eval "use Benchmark ':all'";
+plan skip_all => 'needs Benchmark for testing' if $@;
 
-sub _has_bug_34925 {
-    my %thing;
-    my $r1 = \%thing;
-    my $r2 = \%thing;
-    bless $r1 => 'TestRHBug';
-    return !!$r2;
-}
+plan tests => 3;
 
-sub _possibly_has_bad_overload_performance {
-    return $] < 5.008009 && !_has_bug_34925();
-}
+ok( 1, 'Dummy - prevents next test timing out' );
 
-ok( !_possibly_has_bad_overload_performance(),
-    'Checking not susceptable to bless/overload performance problem' );
+# we do a benchmark test filling an array with blessed/overloaded references,
+# against an array filled with array refs.
+# On a sane system the ratio between these operation sets is 1 - 1.5,
+# whereas a bugged system gives a ratio of around 8
+# we therefore consider there to be a problem if the ratio is >= 2
 
 my $results = timethese(
-    0,
+    -1,    # run for 1 CPU second each
     {
-        overload => sub {
-            use overload q(<) => sub { };
+        no_bless => sub {
             my %h;
-            for ( my $i = 0 ; $i < 5000 ; $i++ ) {
-                $h{$i} = bless [] => 'main';
+            for ( my $i = 0 ; $i < 10000 ; $i++ ) {
+                $h{$i} = [];
             }
         },
-        nooverload => sub {
+        bless_overload => sub {
+            use overload q(<) => sub { };
             my %h;
-            for ( my $i = 0 ; $i < 5000 ; $i++ ) {
+            for ( my $i = 0 ; $i < 10000 ; $i++ ) {
                 $h{$i} = bless [] => 'main';
             }
-          }
-    }
+        },
+    },
 );
 
-ok( ( ( $results->{nooverload}->iters / $results->{overload}->iters ) < 2 ),
-    'Overload/bless performance acceptable' )
+my $ratio = $results->{no_bless}->iters / $results->{bless_overload}->iters;
+
+ok( ( $ratio < 2 ), 'Overload/bless performance acceptable' )
+  || diag(
+    "This perl has a substantial slow down when handling large numbers\n",
+    "of blessed/overloaded objects.  This can severely adversely affect\n",
+    "the performance of DBIx::Class programs.  Please read the section\n",
+    "in the Troubleshooting POD documentation entitled\n",
+    "'Perl Performance Issues on Red Hat Systems'\n",
+  );
+
+# We will only check for the difference in bless handling (whether the
+# bless applies to the reference or the referent) if we have seen a
+# performance issue...
+
+SKIP: {
+    skip "Not checking for bless handling as performance is OK", 1
+      if ( $ratio < 2 );
+
+    {
+        package    # don't want this in PAUSE
+          TestRHBug;
+        use overload bool => sub { 0 }
+    }
+
+    sub _has_bug_34925 {
+        my %thing;
+        my $r1 = \%thing;
+        my $r2 = \%thing;
+        bless $r1 => 'TestRHBug';
+        return !!$r2;
+    }
+
+    sub _possibly_has_bad_overload_performance {
+        return $] < 5.008009 && !_has_bug_34925();
+    }
+
+    # If this next one fails then you almost certainly have a RH derived
+    # perl with the performance bug
+    # if this test fails, look at the section titled
+    # "Perl Performance Issues on Red Hat Systems" in
+    # L<DBIx::Class::Manual::Troubleshooting>
+    # Basically you may suffer severe performance issues when running
+    # DBIx::Class (and many other) modules.  Look at getting a fixed
+    # version of the perl interpreter for your system.
+    #
+    ok( !_possibly_has_bad_overload_performance(),
+        'Checking whether bless applies to reference not object' )
+      || diag(
+        "This perl is probably derived from a buggy Red Hat perl build\n",
+        "Please read the section in the Troubleshooting POD documentation\n",
+        "entitled 'Perl Performance Issues on Red Hat Systems'\n",
+      );
+}