Actual locking and conflict handling

diff --git a/.gitignore b/.gitignore
index d8076a506863cd4f384cff52e4ac5ec34fe9544b..57969882ee41f1edd5f7002d0714ec82fe4b550d 100644 (file)
--- a/.gitignore
+++ b/.gitignore
@@ -12,3 +12,4 @@ yarrg/yarrg
 
 yarrg/_*.*
 yarrg/OCEAN-*.db
+yarrg/Writer.lock

diff --git a/yarrg/CommodsDatabase.pm b/yarrg/CommodsDatabase.pm
index 4866c855d3baacccc7c29ae754a60415169672d6..6f2f6274e09d5b72ddc535d26565eda0757e9acf 100644 (file)
--- a/yarrg/CommodsDatabase.pm
+++ b/yarrg/CommodsDatabase.pm
@@ -22,10 +22,19 @@
 
 package CommodsDatabase;
 
+# Valid calling sequences:
+#    db_setocean('Midnight')
+#  [ db_filename() => 'OCEAN-Midnight.db'  also OK at any later time ]
+#  [ db_writer() ]                         helpful but not essential
+#    db_connect()
+#  [ db_onconflict(sub { .... }) ]         essential if just dieing is not OK
+#    $dbh->do(...), $dbh->prepare(...), db_doall("stmt;stmt;"), etc.
+
 use strict;
 use warnings;
 
 use DBI;
+use POSIX;
 
 use Commods;
 
@@ -34,8 +43,8 @@ BEGIN {
     our ($VERSION, @ISA, @EXPORT, @EXPORT_OK, %EXPORT_TAGS);
     $VERSION     = 1.00;
     @ISA         = qw(Exporter);
-    @EXPORT      = qw(&db_setocean &db_connect $dbh
-                     &db_filename &db_doall);
+    @EXPORT      = qw(&db_setocean &db_writer &db_connect $dbh
+                     &db_filename &db_doall &db_onconflict);
     %EXPORT_TAGS = ( );
 
     @EXPORT_OK   = qw();
@@ -52,12 +61,40 @@ sub db_filename () {
     return $dbfn;
 }
 
+sub db_onconflict (&) {
+    my ($conflictproc) = @_;
+    $dbh->{HandleError}= sub {
+       my ($emsg,$dbh,$val1) = @_;
+       my $native_ecode= $dbh->err();
+       &$conflictproc($emsg) if grep { $_ == $native_ecode } qw(5 6);
+       # 5==SQLITE_BUSY, 6==SQLITE_LOCKED according to the SQLite3
+       # API documentation, .../capi3ref.html#extended-result-codes.
+       return 0; # RaiseError happens next.
+    };
+}
+
+our $writerlockh;
+
+sub db_writer () {
+    my $lockfn= "Writer.lock";
+    $writerlockh= new IO::File "$lockfn", "w" or die "$lockfn $!";
+
+    my $flockall= pack 's!s!LLLLLL', F_WRLCK, SEEK_SET, 0,0,0,0,0,0;
+    # should work everywhere to lock the whole file, provided that
+    # l_type and l_whence are `short int' and come first in that order,
+    # and that start, len and pid are no more than 64 bits each.
+
+    my $r= fcntl($writerlockh, F_SETLKW, $flockall);
+    $r or die "$lockfn fcntl $!";
+}
+
 sub db_connect () {
     $dbh= DBI->connect("dbi:SQLite:$dbfn",'','',
                       { AutoCommit=>0,
                         RaiseError=>1, ShowErrorStatement=>1,
                         unicode=>1 })
        or die "$dbfn $DBI::errstr ?";
+    # default timeout is 30s which is plenty
 }
 
 sub db_doall ($) {

diff --git a/yarrg/commod-email-processor b/yarrg/commod-email-processor
index bf42c0d65ef4b8344f29963de36080b35f664535..ec4f222dcedb25abba947181d5d6a554b2ba9aab 100755 (executable)
--- a/yarrg/commod-email-processor
+++ b/yarrg/commod-email-processor
@@ -117,7 +117,9 @@ sub main () {
     db_setocean($md{'ocean'});
     my $dbfn= db_filename();
     (stat $dbfn) or die "stat database $dbfn failed $!\n";
+    db_writer();
     db_connect();
+    db_onconflict(sub { print STDERR "temporary failure: @_\n"; exit 75; });
 
     my ($islandid) = $dbh->selectrow_array(
               "SELECT islands.islandid