Remove set_txs_connected and set_tx_preconnected.

Author: evoskuil

include/bitcoin/database/impl/query/validate.ipp

@@ -36,16 +36,16 @@ inline code CLASS::to_block_code(
 {
     switch (value)
     {
-        // Block satisfies validation rules (prevouts unverified).
+        // Transitional: Satisfies validation rules (prevouts unverified).
         case schema::block_state::valid:
             return error::block_valid;
-        // Final: Block satisfies confirmation rules (prevouts).
+        // Final: Satisfies confirmation rules (prevouts confirmable).
         case schema::block_state::confirmable:
             return error::block_confirmable;
-        // Final: Block does not satisfy validation/confirmation rules.
+        // Final: Does not satisfy either validation or confirmation rules.
         case schema::block_state::unconfirmable:
             return error::block_unconfirmable;
-        // Block has no recorded state, may be under checkpoint or milestone.
+        // Fault: Has no state, should not happen when read from store.
         default:
             return error::unknown_state;
     }
@@ -61,16 +61,13 @@ inline code CLASS::to_tx_code(
     // All states below are relevant only to the associated validation context.
     switch (value)
     {
-        // Tx is valid in the case where standard prevouts are matched.
-        case schema::tx_state::preconnected:
-            return error::tx_preconnected;
-        // Final: Tx is valid (passed check, accept, and connect).
+        // Final: Is valid (passed check, accept, and connect).
         case schema::tx_state::connected:
             return error::tx_connected;
-        // Final: Tx is not valid (failed check, accept, or connect).
+        // Final: Is not valid (failed check, accept, or connect).
         case schema::tx_state::disconnected:
             return error::tx_disconnected;
-        // Tx has no recorded state, may be under checkpoint or milestone.
+        // Fault: Has no state, should not happen when read from store.
         default:
             return error::unknown_state;
     }
@@ -307,25 +304,6 @@ bool CLASS::set_block_unconfirmable(const header_link& link) NOEXCEPT
     // ========================================================================
 }
 
-TEMPLATE
-bool CLASS::set_tx_preconnected(const tx_link& link,
-    const context& ctx) NOEXCEPT
-{
-    // ========================================================================
-    const auto scope = store_.get_transactor();
-
-    // Clean single allocation failure (e.g. disk full).
-    return store_.validated_tx.put(link, table::validated_tx::slab
-    {
-        {},
-        ctx,
-        schema::tx_state::preconnected,
-        0, // fee
-        0  // sigops
-    });
-    // ========================================================================
-}
-
 TEMPLATE
 bool CLASS::set_tx_disconnected(const tx_link& link,
     const context& ctx) NOEXCEPT
@@ -367,40 +345,40 @@ bool CLASS::set_tx_connected(const tx_link& link, const context& ctx,
     // ========================================================================
 }
 
-TEMPLATE
-bool CLASS::set_txs_connected(const header_link& link) NOEXCEPT
-{
-    context ctx{};
-    if (!get_context(ctx, link))
-        return false;
-
-    const auto txs = to_transactions(link);
-    if (txs.empty())
-        return false;
-
-    // FOR PERFORMANCE EVALUATION ONLY.
-    constexpr uint64_t fee = 99;
-    constexpr size_t sigops = 42;
-    using sigs = linkage<schema::sigops>;
-
-    // ========================================================================
-    const auto scope = store_.get_transactor();
-
-    // Clean single allocation failure (e.g. disk full).
-    return std_all_of(bc::seq, txs.begin(), txs.end(),
-        [&](const tx_link& fk) NOEXCEPT
-        {
-            return store_.validated_tx.put(fk, table::validated_tx::slab
-            {
-                {},
-                ctx,
-                schema::tx_state::connected,
-                fee,
-                system::possible_narrow_cast<sigs::integer>(sigops)
-            });
-        });
-    // ========================================================================
-}
+////TEMPLATE
+////bool CLASS::set_txs_connected(const header_link& link) NOEXCEPT
+////{
+////    context ctx{};
+////    if (!get_context(ctx, link))
+////        return false;
+////
+////    const auto txs = to_transactions(link);
+////    if (txs.empty())
+////        return false;
+////
+////    // FOR PERFORMANCE EVALUATION ONLY.
+////    constexpr uint64_t fee = 99;
+////    constexpr size_t sigops = 42;
+////    using sigs = linkage<schema::sigops>;
+////
+////    // ========================================================================
+////    const auto scope = store_.get_transactor();
+////
+////    // Clean single allocation failure (e.g. disk full).
+////    return std_all_of(bc::seq, txs.begin(), txs.end(),
+////        [&](const tx_link& fk) NOEXCEPT
+////        {
+////            return store_.validated_tx.put(fk, table::validated_tx::slab
+////            {
+////                {},
+////                ctx,
+////                schema::tx_state::connected,
+////                fee,
+////                system::possible_narrow_cast<sigs::integer>(sigops)
+////            });
+////        });
+////    // ========================================================================
+////}
 
 } // namespace database
 } // namespace libbitcoin

include/bitcoin/database/query.hpp

@@ -478,10 +478,6 @@ class query
     bool set_block_valid(const header_link& link, uint64_t fees) NOEXCEPT;
     bool set_block_unconfirmable(const header_link& link) NOEXCEPT;
     bool set_block_confirmable(const header_link& link) NOEXCEPT;
-
-    // set_txs_connected is FOR PERFORMANCE EVALUATION ONLY.
-    bool set_txs_connected(const header_link& link) NOEXCEPT;
-    bool set_tx_preconnected(const tx_link& link, const context& ctx) NOEXCEPT;
     bool set_tx_disconnected(const tx_link& link, const context& ctx) NOEXCEPT;
     bool set_tx_connected(const tx_link& link, const context& ctx,
         uint64_t fee, size_t sigops) NOEXCEPT;

include/bitcoin/database/tables/schema.hpp

@@ -58,7 +58,6 @@ namespace schema
         constexpr auto candidate = "candidate";
         constexpr auto confirmed = "confirmed";
         constexpr auto strong_tx = "strong_tx";
-        ////constexpr auto spent_out = "spent_out";
     }
 
     namespace caches
@@ -89,16 +88,15 @@ namespace schema
 
     enum block_state : uint8_t
     {
-        confirmable = 0,    // final
-        valid = 1, // transitional
-        unconfirmable = 2   // final
+        confirmable = 0,            // final
+        valid = 1,                  // transitional
+        unconfirmable = 2           // final
     };
 
     enum tx_state : uint8_t
     {
-        connected = 0,      // final
-        preconnected = 1,   // transitional
-        disconnected = 2    // final
+        connected = 0,              // final
+        disconnected = 1            // final
     };
 
     /// Values.

test/query/validate.cpp

@@ -349,28 +349,6 @@ BOOST_AUTO_TEST_CASE(query_validate__get_tx_state__connected_in_context__tx_conn
     BOOST_REQUIRE_EQUAL(sigops, expected_sigops);
 }
 
-BOOST_AUTO_TEST_CASE(query_validate__get_tx_state__connected_in_context__tx_preconnected)
-{
-    settings settings{};
-    settings.path = TEST_DIRECTORY;
-    test::chunk_store store{ settings };
-    test::query_accessor query{ store };
-    BOOST_REQUIRE_EQUAL(store.create(events_handler), error::success);
-    BOOST_REQUIRE(query.initialize(test::genesis));
-    BOOST_REQUIRE(query.set(test::block1, context{}, false, false));
-    BOOST_REQUIRE(query.set(test::block2, context{}, false, false));
-    BOOST_REQUIRE(query.set(test::block3, context{}, false, false));
-
-    uint64_t fee{};
-    size_t sigops{};
-    constexpr context ctx{ 7, 8, 9 };
-    BOOST_REQUIRE(query.set_tx_preconnected(3, ctx));
-    BOOST_REQUIRE_EQUAL(query.get_tx_state(3, ctx), error::tx_preconnected);
-    BOOST_REQUIRE_EQUAL(query.get_tx_state(fee, sigops, 3, ctx), error::tx_preconnected);
-    BOOST_REQUIRE_EQUAL(fee, 0u);
-    BOOST_REQUIRE_EQUAL(sigops, 0u);
-}
-
 BOOST_AUTO_TEST_CASE(query_validate__get_tx_state__connected_in_context__tx_disconnected)
 {
     settings settings{};