Changeset 19155 for trunk

Timestamp:

08/12/18 15:48:05 (6 years ago)

Author:

tbretz

Message:

Added the --index option.

Location:

trunk/FACT++/src

Files:

• fits2sql.cc (modified) (6 diffs)
• root2sql.cc (modified) (7 diffs)

trunk/FACT++/src/fits2sql.cc

@@ -68 +68 @@
         ("conditional",    po_switch(),               "Conditional insert. Only insert if no entry exists yet with the constants defined by --const")
         ("delete",         po_switch(),               "Delete all entries first which fit all constant columns defined by --const")
+        ("index",          po_switch(),               "If a table is created, all const columns are used as a single index")
         ;
 
@@ -197 +198 @@
         "If any row with the given constant values are found, the execution is stopped "
         "(note that this happend after the table drop/create but before the delete/insert.\n"
+        "\n"
+        "To ensure efficient access for a conditonal execution, it makes sense to have "
+        "an index created for those columns. This can be done during table creation "
+        "with the --index option.\n"
         "\n"
         "If a query failed, the query is printed to stderr together with the error message. "
@@ -324 +329 @@
     const bool conditional       = conf.Get<bool>("conditional");
     const bool run_delete        = conf.Get<bool>("delete");
+    const bool index             = conf.Get<bool>("index");
 
     const string engine          = conf.Get<string>("engine");
@@ -412 +418 @@
 
     string where;
+    vector<string> vindex;
     for (auto it=fixed.cbegin(); it!=fixed.cend(); it++)
     {
@@ -450 +457 @@
         vec.emplace_back(name, val);
         where += " AND `"+name+"`="+val;
+        vindex.emplace_back(name);
     }
 
@@ -581 +589 @@
         query += ",\n   PRIMARY KEY USING BTREE (`"+boost::algorithm::join(primary, "`, `")+"`)";
 
+    if (!vindex.empty() && index)
+        query += ",\n   INDEX USING BTREE (`"+boost::algorithm::join(vindex, "`, `")+"`)";
+
     query +=
         "\n)\n"

trunk/FACT++/src/root2sql.cc

@@ -71 +71 @@
         ("conditional",    po_switch(),               "Conditional insert. Only insert if no entry exists yet with the constants defined by --const")
         ("delete",         po_switch(),               "Delete all entries first which fit all constant columns defined by --const")
+        ("index",          po_switch(),               "If a table is created, all const columns are used as a single index")
         ;
 
@@ -205 +206 @@
         "(note that this happend after the table drop/create but before the delete/insert.\n"
         "\n"
+        "To ensure efficient access for a conditonal execution, it makes sense to have "
+        "an index created for those columns. This can be done during table creation "
+        "with the --index option.\n"
+        "\n"
         "If a query failed, the query is printed to stderr together with the error message. "
         "For the main INSERT query, this is only true if the verbosity level is at least 2 "
@@ -319 +324 @@
     if (string(msg).substr(0,24)=="no dictionary for class ")
         return;
+    if (string(msg).substr(0, 5)=="unknown branch ")
+        return;
 
     DefaultErrorHandler(level, abort, location, msg);
@@ -353 +360 @@
     const bool conditional       = conf.Get<bool>("conditional");
     const bool run_delete        = conf.Get<bool>("delete");
+    const bool index             = conf.Get<bool>("index");
 
     const string engine          = conf.Get<string>("engine");
@@ -435 +443 @@
 
     string where;
+    vector<string> vindex;
     for (auto it=fixed.cbegin(); it!=fixed.cend(); it++)
     {
@@ -473 +482 @@
         vec.emplace_back(name, val);
         where += " AND `"+name+"`="+val;
+        vindex.emplace_back(name);
     }
 
@@ -615 +625 @@
     if (!primary.empty())
         query += ",\n   PRIMARY KEY USING BTREE (`"+boost::algorithm::join(primary, "`, `")+"`)";
+
+    if (!vindex.empty() && index)
+        query += ",\n   INDEX USING BTREE (`"+boost::algorithm::join(vindex, "`, `")+"`)";
 
     query +=