Changeset 31188 for main/trunk/greenstone2/perllib/DBDrivers

Timestamp:

2016-12-09T21:24:55+13:00 (7 years ago)

Author:

ak19

Message:

This commit is related to but not specific to the upcoming commit to do with the new oaiinfo db and its directly affected files. This commit: New remove and rename (move) methods in DB package to clean up main db file and any additional db files created by any specific infodbtype. The new methods in dbutil/jdbm.pm are not called, for some reason. Requires more investigation, but committing for now.

Location:

main/trunk/greenstone2/perllib/DBDrivers

Files:

• BaseDBDriver.pm (modified) (1 diff)
• GDBM.pm (modified) (1 diff)
• JDBM.pm (modified) (2 diffs)

main/trunk/greenstone2/perllib/DBDrivers/BaseDBDriver.pm

@@ -344 +344 @@
 
 
+## @function rename_db_file_to(string, string) => void
+#
+sub rename_db_file_to {
+    my $self = shift(@_);
+    my ($srcpath, $destpath) = @_;
+
+    # rename basic db file
+    &FileUtils::moveFiles($srcpath, $destpath);
+
+    # subclass should rename any additional files that the specific dbtype creates
+}
+## rename_db_file_to(string, string) => void ##
+
+## @function remove_db_file(string) => void
+#
+sub remove_db_file {
+    my $self = shift(@_);
+    my ($db_filepath) = @_;
+    
+    # remove basic db file
+    &FileUtils::removeFiles($db_filepath);
+
+    # subclass must rename any additional files that the specific dbtype creates (e.g. transaction log files)
+}
+## remove_db_file(string, string) => void ##
+
+
 ## @function supportsDatestamp(void) => integer
 #

main/trunk/greenstone2/perllib/DBDrivers/GDBM.pm

@@ -70 +70 @@
 # Handled by BaseDBDriver
 # sub get_infodb_file_path(string, string) => string
+# sub rename_db_file_to(string, string) => void
+# sub remove_db_file(string) => void
 
 # Handled by 70HyphenFormat

main/trunk/greenstone2/perllib/DBDrivers/JDBM.pm

@@ -83 +83 @@
 # -----------------------------------------------------------------------------
 
-# When DBUtil::* is properly structured with inheritence, then
+# When DBUtil::* is properly structured with inheritance, then
 # much of this code (along with GDBM and GDBM-TXT-GZ) can be grouped into
 # a shared base class.  Really it is only the the command that needs to
@@ -91 +91 @@
 # sub get_infodb_file_path {}
 
-# Handles by 70HyphenFormat
+# Handled by 70HyphenFormat
 # sub open_infodb_write_handle(string, string?) => filehandle
 # sub close_infodb_write_handle(filehandle) => void