fix: address narrowing conversions and missing includes in server and transaction manager

Author: poyrazK

src/network/server.cpp

@@ -360,13 +360,13 @@ void Server::handle_connection(int client_fd) {
                         // Row Description (T)
                         if (!res.rows().empty() && res.schema().column_count() > 0) {
                             const auto& schema = res.schema();
-                            const uint32_t num_cols = static_cast<uint32_t>(schema.column_count());
+                            const auto num_cols = static_cast<uint32_t>(schema.column_count());
 
                             // Calculate T packet length
                             uint32_t t_len = 4 + 2;  // len + num_cols
                             for (uint32_t i = 0; i < num_cols; ++i) {
-                                t_len +=
-                                    schema.get_column(i).name().size() + 1 + 4 + 2 + 4 + 2 + 4 + 2;
+                                t_len += static_cast<uint32_t>(schema.get_column(i).name().size()) +
+                                         1 + 4 + 2 + 4 + 2 + 4 + 2;
                             }
 
                             const char t_type = 'T';

src/transaction/transaction_manager.cpp

@@ -6,13 +6,16 @@
 #include "transaction/transaction_manager.hpp"
 
 #include <algorithm>
+#include <cstddef>
 #include <memory>
 #include <mutex>
 #include <utility>
 
 #include "catalog/catalog.hpp"
+#include "executor/types.hpp"
 #include "recovery/log_manager.hpp"
 #include "recovery/log_record.hpp"
+#include "storage/buffer_pool_manager.hpp"
 #include "storage/heap_table.hpp"
 #include "transaction/lock_manager.hpp"
 #include "transaction/transaction.hpp"
@@ -78,7 +81,7 @@ void TransactionManager::commit(Transaction* txn) {
         completed_transactions_.push_back(std::move(active_transactions_[txn->get_id()]));
         static_cast<void>(active_transactions_.erase(txn->get_id()));
 
-        constexpr size_t MAX_COMPLETED = 100;
+        constexpr std::size_t MAX_COMPLETED = 100;
         if (completed_transactions_.size() > MAX_COMPLETED) {
             completed_transactions_.pop_front();
         }
@@ -113,7 +116,7 @@ void TransactionManager::abort(Transaction* txn) {
         completed_transactions_.push_back(std::move(active_transactions_[txn->get_id()]));
         static_cast<void>(active_transactions_.erase(txn->get_id()));
 
-        constexpr size_t MAX_COMPLETED = 100;
+        constexpr std::size_t MAX_COMPLETED = 100;
         if (completed_transactions_.size() > MAX_COMPLETED) {
             completed_transactions_.pop_front();
         }

tests/cloudSQL_tests.cpp

@@ -45,7 +45,7 @@ constexpr double PI_UPPER = 3.15;
 constexpr int64_t VAL_1 = 1;
 constexpr int64_t VAL_2 = 2;
 constexpr int64_t VAL_10 = 10;
-
+constexpr int64_t VAL_20 = 20;
 constexpr int64_t VAL_25 = 25;
 constexpr uint64_t STATS_100 = 100;
 constexpr uint16_t PORT_5432 = 5432;