history DB: Add "persistence" column

For rpm-software-management/dnf#2196.
The `persistence` column is a TransactionPersistence enum, currently it
can either be UNKNOWN, PERSIST, or TRANSIENT.

Author: evan-goode

libdnf/transaction/Swdb.cpp

@@ -385,6 +385,14 @@ Swdb::setReleasever(std::string value)
     transactionInProgress->setReleasever(value);
 }
 
+void
+Swdb::setPersistence(TransactionPersistence persistence)
+{
+    if (!transactionInProgress) {
+        throw std::logic_error(_("Not in progress"));
+    }
+    transactionInProgress->setPersistence(persistence);
+}
 
 void
 Swdb::addConsoleOutputLine(int fileDescriptor, std::string line)

libdnf/transaction/Swdb.hpp

@@ -114,6 +114,7 @@ struct Swdb {
 
     // misc
     void setReleasever(std::string value);
+    void setPersistence(TransactionPersistence value);
     void addConsoleOutputLine(int fileDescriptor, std::string line);
 
     /**

libdnf/transaction/Transaction.cpp

@@ -82,6 +82,7 @@ Transaction::dbSelect(int64_t pk)
         "  releasever, "
         "  user_id, "
         "  cmdline, "
+        "  persistence, "
         "  state, "
         "  comment "
         "FROM "
@@ -100,6 +101,7 @@ Transaction::dbSelect(int64_t pk)
     releasever = query.get< std::string >("releasever");
     userId = query.get< uint32_t >("user_id");
     cmdline = query.get< std::string >("cmdline");
+    persistence = static_cast<TransactionPersistence>(query.get<int>("persistence"));
     state = static_cast< TransactionState >(query.get< int >("state"));
     comment = query.get< std::string >("comment");
 }

libdnf/transaction/Transaction.hpp

@@ -55,6 +55,8 @@ class Transaction {
     const std::string &getReleasever() const noexcept { return releasever; }
     uint32_t getUserId() const noexcept { return userId; }
     const std::string &getCmdline() const noexcept { return cmdline; }
+    TransactionPersistence getPersistence() const noexcept { return persistence; }
+
     TransactionState getState() const noexcept { return state; }
     const std::string &getComment() const noexcept { return comment; }
 
@@ -79,6 +81,7 @@ class Transaction {
     std::string releasever;
     uint32_t userId = 0;
     std::string cmdline;
+    TransactionPersistence persistence = TransactionPersistence::UNKNOWN;
     TransactionState state = TransactionState::UNKNOWN;
     std::string comment;
 };

libdnf/transaction/Transformer.cpp

@@ -53,6 +53,10 @@ static const char * const sql_migrate_tables_1_2 =
 #include "sql/migrate_tables_1_2.sql"
     ;
 
+static const char * const sql_migrate_tables_1_3 =
+#include "sql/migrate_tables_1_3.sql"
+    ;
+
 void
 Transformer::createDatabase(SQLite3Ptr conn)
 {
@@ -71,6 +75,9 @@ Transformer::migrateSchema(SQLite3Ptr conn)
         if (schemaVersion == "1.1") {
             conn->exec(sql_migrate_tables_1_2);
         }
+        if (schemaVersion == "1.2") {
+            conn->exec(sql_migrate_tables_1_3);
+        }
     }
     else {
         throw Exception(_("Database Corrupted: no row 'version' in table 'config'"));

libdnf/transaction/Types.hpp

@@ -56,6 +56,12 @@ enum class TransactionItemAction : int {
     REASON_CHANGE = 11 // a package was kept on the system but it's reason has changed
 };
 
+enum class TransactionPersistence : int {
+    UNKNOWN = 0,
+    PERSIST = 1,
+    TRANSIENT = 2,
+};
+
 } // namespace libdnf
 /*
 Install

libdnf/transaction/private/Transaction.cpp

@@ -76,12 +76,13 @@ swdb_private::Transaction::dbInsert()
         "    releasever, "
         "    user_id, "
         "    cmdline, "
+        "    persistence, "
         "    state, "
         "    comment, "
         "    id "
         "  ) "
         "VALUES "
-        "  (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
+        "  (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
     SQLite3::Statement query(*conn.get(), sql);
     query.bindv(getDtBegin(),
                 getDtEnd(),
@@ -90,10 +91,11 @@ swdb_private::Transaction::dbInsert()
                 getReleasever(),
                 getUserId(),
                 getCmdline(),
+                static_cast<int>(getPersistence()),
                 static_cast< int >(getState()),
                 getComment());
     if (getId() > 0) {
-        query.bind(9, getId());
+        query.bind(10, getId());
     }
     query.step();
     setId(conn->lastInsertRowID());
@@ -138,6 +140,7 @@ swdb_private::Transaction::dbUpdate()
         "  releasever=?, "
         "  user_id=?, "
         "  cmdline=?, "
+        "  persistence=?, "
         "  state=?, "
         "  comment=? "
         "WHERE "
@@ -150,6 +153,7 @@ swdb_private::Transaction::dbUpdate()
                 getReleasever(),
                 getUserId(),
                 getCmdline(),
+                static_cast<int>(getPersistence()),
                 static_cast< int >(getState()),
                 getComment(),
                 getId());

libdnf/transaction/private/Transaction.hpp

@@ -42,6 +42,7 @@ class Transaction : public libdnf::Transaction {
     void setReleasever(const std::string &value) { releasever = value; }
     void setUserId(uint32_t value) { userId = value; }
     void setCmdline(const std::string &value) { cmdline = value; }
+    void setPersistence(TransactionPersistence value) { persistence = value; }
     void setState(TransactionState value) { state = value; }
     void setComment(const std::string &value) { comment = value; }
 

libdnf/transaction/sql/create_tables.sql

@@ -8,6 +8,7 @@ R"**(
         releasever TEXT NOT NULL,       /* var: $releasever */
         user_id INTEGER NOT NULL,       /* user ID (UID) */
         cmdline TEXT,                   /* recorded command line (program, options, arguments) */
+        persistence INTEGER DEFAULT 0   /* transaction persistence: unknown, persist, or transient
         state INTEGER NOT NULL          /* (enum) */
     );
     CREATE TABLE repo (