/[wait]/trunk/lib/WAIT/Table.pm

This is repository of my old source code which isn't updated any more. Go to git.rot13.org for current projects!

Diff of /trunk/lib/WAIT/Table.pm

Parent Directory | Revision Log | View Patch Patch

-cvs-head/lib/WAIT/Table.pm
revision 34 by ulpfr,
Sun Nov 12 14:22:40 2000 UTC
+trunk/lib/WAIT/Table.pm
revision 89 by dpavlin,
Mon May 24 20:57:08 2004 UTC
 Line 4
  # Author          : Ulrich Pfeifer
  # Created On      : Thu Aug  8 13:05:10 1996
  # Last Modified By: Ulrich Pfeifer
- # Last Modified On: Sun Nov 12 15:21:19 2000
+ # Last Modified On: Wed Jan 23 14:15:15 2002
  # Language        : CPerl
- # Update Count    : 135
+ # Update Count    : 152
  # Status          : Unknown, Use with caution!
  #
  # Copyright (c) 1996-1997, Ulrich Pfeifer
 Line 165 
 sub new {
      croak "Could not 'mkdir $self->{file}': $!\n";
    }
-   my $lockmgr = LockFile::Simple->make(-autoclean => 1);
-   # Aquire a write lock, since we are creating the table, no readers
-   # could possibly be active.
-   $self->{write_lock} = $lockmgr->lock($self->{file} . '/write')
-     or die "Can't lock '$self->{file}/write'";
    $self->{djk}      = $parm{djk}      if defined $parm{djk};
    $self->{layout}   = $parm{layout} || new WAIT::Parse::Base;
    $self->{access}   = $parm{access} if defined $parm{access};
-Line 179 
 sub new {
+Line 173 
 sub new {
    $self->{indexes}  = {};
    bless $self, $type;
+   # Checking for readers is not necessary, but let's go with the
+   # generic method.
+   $self->getlock(O_RDWR|O_CREAT); # dies when failing
    # Call create_index() and create_index() for compatibility
    for (@{$self->{keyset}||[]}) {
      #carp "Specification of indexes at table create time is deprecated";
-Line 324 
 Must be called via C<WAIT::Database::dro
+Line 323 
 Must be called via C<WAIT::Database::dro
  sub drop {
    my $self = shift;
+   unless ($self->{write_lock}){
+     warn "Cannot drop table without write lock. Nothing done";
+     return;
+   }
    if ((caller)[0] eq 'WAIT::Database') { # database knows about this
      $self->close;               # just make sure
      my $file = $self->{file};
      for (values %{$self->{indexes}}) {
        $_->drop;
      }
      unlink "$file/records";
+     rmdir "$file/read" or warn "Could not rmdir '$file/read'";
      # $self->unlock;
      ! (!-e $file or rmdir $file);
    } else {
-Line 373 
 sub open {
+Line 381 
 sub open {
      }
      require WAIT::InvertedIndex;
    }
+   $self->getlock($self->{mode});
    unless (defined $self->{dbh}) {
      if ($USE_RECNO) {
        $self->{dbh} = tie(@{$self->{db}}, 'DB_File', $file,
-Line 383 
 sub open {
+Line 394 
 sub open {
                           $self->{mode}, 0664, $DB_BTREE);
      }
    }
-   # Locking
-   #
-   # We allow multiple readers to coexists.  But write access excludes
-   # all read access and vice versa.  In practice read access on tables
-   # open for writing will mostly work ;-)
-   my $lockmgr = LockFile::Simple->make(-autoclean => 1);
-   my $lockdir = $self->{file} . '/read';
-   unless (-d $lockdir) {
-     mkdir $lockdir, 0755 or die "Could not mkdir $lockdir: $!";
-   }
-   if ($self->{mode} & O_RDWR) {
-     # Get a write lock.  Release it again and die if there is any
-     # valid reader.
-     # this is a hack.  We do not check for reopening ...
-     return $self if $self->{write_lock};
-     if ($self->{read_lock}) {
-       # We are a becoming a writer now. So we release the read lock to
-       # avoid blocking ourselves.
-       $self->{read_lock}->release;
-       delete $self->{read_lock};
-     }
-     # Get the preliminary write lock
-     $self->{write_lock} = $lockmgr->lock($self->{file} . '/write')
-       or die "Can't lock '$self->{file}/write'";
-     # If we actually want to write we must check if there are any
-     # readers.  The write lock is confirmed if wen cannot find any
-     # valid readers.
-     local *DIR;
-     opendir DIR, $lockdir or
-       die "Could not opendir '$lockdir': $!";
-     for my $lockfile (grep { -f "$lockdir/$_" } readdir DIR) {
-       # check if the locks are still valid.
-       # Since we are protected by a write lock, we could use a plain file.
-       # But we want to use the stale testing from LockFile::Simple.
-       if (my $lck = $lockmgr->trylock("$lockdir/$lockfile")) {
-         warn "Removing stale lockfile '$lockdir/$lockfile'";
-         $lck->release;
-       } else {
-         $self->{write_lock}->release;
-         die "Cannot write table '$file' while it's in use";
-       }
-     }
-     closedir DIR;
-   } else {
-     # this is a hack.  We do not check for reopening ...
-     return $self if $self->{read_lock};
-     # Get the preliminary write lock to protect the directory
-     # operations.
-     $self->{write_lock} ||= $lockmgr->lock($self->{file} . '/write')
-       or die "Can't lock '$self->{file}/write'";
-     # find a new read slot
-     my $id = time;
-     while (-f "$lockdir/$id.lock") { # here assume ".lock" format!
-       $id++;
-     }
-     $self->{read_lock} = $lockmgr->lock("$lockdir/$id")
-       or die "Can't lock '$lockdir/$id'";
-     # We are a reader now. So we release the write lock
-     $self->{write_lock}->release;
-     delete $self->{write_lock};
-   }
    $self;
  }
-Line 660 
 sub set {
+Line 597 
 sub set {
      warn "Cannot set iattr[$iattr] without write lock. Nothing done";
      return;
    }
+   # in the rare case that they haven't written a single record yet, we
+   # make sure, the inverted inherits our $self->{mode}:
+   defined $self->{db} or $self->open;
    for my $att (keys %{$self->{inverted}}) {
-     if ($] > 5.003) {         # avoid bug in perl up to 5.003_05
+     require WAIT::InvertedIndex;
+     if ($^V gt v5.003) {         # avoid bug in perl up to 5.003_05
        my $idx;
        for $idx (@{$self->{inverted}->{$att}}) {
          $idx->set($iattr, $value);
-Line 717 
 sub close {
+Line 660 
 sub close {
 ;
  }
+ # Locking
+ #
+ # We allow multiple readers to coexists.  But write access excludes
+ # all read access and vice versa.  In practice read access on tables
+ # open for writing will mostly work ;-)
+ # If a "write" lock is requested, an existing "read" lock will be
+ # released.  If a "read" lock ist requested, an existing "write" lock
+ # will be released.  Requiring a lock already hold has no effect.
+ sub getlock {
+   my ($self, $mode) = @_;
+   # autoclean cleans on DESTROY, stale sends SIGZERO to the owner
+   #
+   my $lockmgr = LockFile::Simple->make(-autoclean => 1, -stale => 1);
+   my $file    = $self->{file} . '/records';
+   my $lockdir = $self->{file} . '/read';
+   unless (-d $lockdir) {
+     mkdir $lockdir, 0755 or die "Could not mkdir $lockdir: $!";
+   }
+   if ($mode & O_RDWR) {         # Get a write lock.  Release it again
+                                 # and die if there is any valid
+                                 # readers.
+     # Have a write lock already
+     return $self if $self->{write_lock};
+     if ($self->{read_lock}) {   # We are a becoming a writer now. So
+                                 # we release the read lock to avoid
+                                 # blocking ourselves.
+       $self->{read_lock}->release;
+       delete $self->{read_lock};
+     }
+     # Get the preliminary write lock
+     $self->{write_lock} = $lockmgr->lock($self->{file} . '/write')
+       or die "Can't lock '$self->{file}/write'";
+     # If we actually want to write we must check if there are any
+     # readers.  The write lock is confirmed if wen cannot find any
+     # valid readers.
+     local *DIR;
+     opendir DIR, $lockdir or
+       die "Could not opendir '$lockdir': $!";
+     for my $lockfile (grep { -f "$lockdir/$_" } readdir DIR) {
+       # Check if the locks are still valid.  Since we are protected by
+       # a write lock, we could use a plain file.  But we want to use
+       # the stale testing from LockFile::Simple.
+       if (my $lck = $lockmgr->trylock("$lockdir/$lockfile")) {
+         warn "Removing stale lockfile '$lockdir/$lockfile'";
+         $lck->release;
+       } else {                  # Found an active reader, rats!
+         $self->{write_lock}->release;
+         die "Cannot write table '$file' while it's in use";
+       }
+     }
+     closedir DIR;
+   } else {
+     # Have a read lock already
+     return $self if $self->{read_lock};
+     # Get the preliminary write lock to protect the directory
+     # operations.
+     my $write_lock = $lockmgr->lock($self->{file} . '/read/write')
+       or die "Can't lock '$self->{file}/read/write'";
+     # Find a new read slot.  Maybe the plain file would be better?
+     my $id = time;
+     while (-f "$lockdir/$id.lock") { # here assume ".lock" format!
+       $id++;
+     }
+     $self->{read_lock} = $lockmgr->lock("$lockdir/$id")
+       or die "Can't lock '$lockdir/$id'";
+     # We are a reader now. So we release the write lock
+     $write_lock->release;
+   }
+   return $self;
+ }
  sub unlock {
    my $self = shift;
-Line 738 
 sub unlock {
+Line 767 
 sub unlock {
  sub DESTROY {
    my $self = shift;
-   warn "Table handle destroyed without closing it first"
+   if ($self->{write_lock} || $self->{read_lock}) {
-     if $self->{write_lock} || $self->{read_lock};
+     warn "Table handle destroyed without closing it first";
+     $self->unlock;
+   }
  }
  sub open_scan {

 Legend:



Removed from v.34
 


changed lines


 
Added in v.89
 Legend:



Removed from v.34
 


changed lines


 
Added in v.89
-Removed from v.34
+Added in v.89

	ViewVC Help
Powered by ViewVC 1.1.26