chore: optimize CI pipeline and add automated formatting

Summary:
- Switched to Ninja build system for speed.
- Implemented build matrix for Clang and GCC.
- Enabled ccache for faster compilation.
- Added automated clang-format style check.
- Restyled entire codebase using Google C++ style.

Author: poyrazK

.clang-format

@@ -0,0 +1,8 @@
+---
+Language:        Cpp
+BasedOnStyle:    Google
+IndentWidth:     4
+ColumnLimit:     100
+AllowShortFunctionsOnASingleLine: Inline
+KeepEmptyLinesAtTheStartOfBlocks: false
+...

.github/workflows/ci.yml

@@ -7,28 +7,55 @@ on:
     branches: [ main ]
 
 jobs:
+  style-check:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Run clang-format check
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y clang-format
+        find src include tests -name "*.cpp" -o -name "*.hpp" | xargs clang-format --dry-run --Werror
+
   build-and-test:
+    needs: style-check
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        compiler: [clang++, g++]
 
     steps:
     - uses: actions/checkout@v3
 
     - name: Install dependencies
       run: |
         sudo apt-get update
-        sudo apt-get install -y cmake clang clang-tidy llvm
+        sudo apt-get install -y cmake clang clang-tidy llvm ninja-build ccache
+
+    - name: Cache ccache
+      uses: actions/cache@v3
+      with:
+        path: ~/.ccache
+        key: ${{ runner.os }}-${{ matrix.compiler }}-ccache-${{ github.sha }}
+        restore-keys: |
+          ${{ runner.os }}-${{ matrix.compiler }}-ccache-
 
     - name: Configure CMake
       run: |
         mkdir build
         cd build
-        cmake .. -DCMAKE_CXX_COMPILER=clang++ -DBUILD_TESTS=ON -DBUILD_COVERAGE=OFF
+        export CCACHE_DIR=~/.ccache
+        cmake .. -G Ninja \
+          -DCMAKE_CXX_COMPILER=${{ matrix.compiler }} \
+          -DCMAKE_CXX_COMPILER_LAUNCHER=ccache \
+          -DBUILD_TESTS=ON \
+          -DBUILD_COVERAGE=OFF
 
-    - name: Build with Clang-Tidy
+    - name: Build
       run: |
         cd build
-        make -j$(nproc) sqlEngineCore
-        make -j$(nproc)
+        ninja sqlEngineCore
+        ninja
 
     - name: Run Tests
       run: |
@@ -39,7 +66,3 @@ jobs:
         ./transaction_manager_tests
         ./statement_tests
         ./recovery_tests
-
-    - name: Check memory safety (ASan)
-      run: |
-        echo "ASan is enabled via CMAKE_CXX_FLAGS in CMakeLists.txt and verified during 'Run Tests' step."

include/catalog/catalog.hpp

@@ -8,14 +8,15 @@
 
 #include <cstdint>
 #include <cstring>
-#include <string>
-#include <vector>
-#include <unordered_map>
+#include <fstream>
+#include <iostream>
 #include <memory>
 #include <optional>
+#include <string>
+#include <unordered_map>
 #include <variant>
-#include <fstream>
-#include <iostream>
+#include <vector>
+
 #include "common/value.hpp"
 
 namespace cloudsql {
@@ -27,7 +28,7 @@ using oid_t = uint32_t;
  * @brief Column information structure (C++ class)
  */
 class ColumnInfo {
-public:
+   public:
     std::string name;
     common::ValueType type;
     uint16_t position;
@@ -37,43 +38,34 @@ class ColumnInfo {
     std::optional<std::string> default_value;
     uint32_t flags;
 
-    ColumnInfo() 
-        : type(common::TYPE_NULL)
-        , position(0)
-        , max_length(0)
-        , nullable(true)
-        , is_primary_key(false)
-        , flags(0)
-    {}
+    ColumnInfo()
+        : type(common::TYPE_NULL),
+          position(0),
+          max_length(0),
+          nullable(true),
+          is_primary_key(false),
+          flags(0) {}
 
     ColumnInfo(std::string name, common::ValueType type, uint16_t pos)
-        : name(std::move(name))
-        , type(type)
-        , position(pos)
-        , max_length(0)
-        , nullable(true)
-        , is_primary_key(false)
-        , flags(0)
-    {}
+        : name(std::move(name)),
+          type(type),
+          position(pos),
+          max_length(0),
+          nullable(true),
+          is_primary_key(false),
+          flags(0) {}
 };
 
 /**
  * @brief Index type enumeration
  */
-enum class IndexType : uint8_t {
-    BTree = 0,
-    Hash = 1,
-    GiST = 2,
-    SPGiST = 3,
-    GIN = 3,
-    BRIN = 4
-};
+enum class IndexType : uint8_t { BTree = 0, Hash = 1, GiST = 2, SPGiST = 3, GIN = 3, BRIN = 4 };
 
 /**
  * @brief Index information structure (C++ class)
  */
 class IndexInfo {
-public:
+   public:
     oid_t index_id;
     std::string name;
     oid_t table_id;
@@ -85,20 +77,19 @@ class IndexInfo {
     uint32_t flags;
 
     IndexInfo()
-        : index_id(0)
-        , table_id(0)
-        , index_type(IndexType::BTree)
-        , is_unique(false)
-        , is_primary(false)
-        , flags(0)
-    {}
+        : index_id(0),
+          table_id(0),
+          index_type(IndexType::BTree),
+          is_unique(false),
+          is_primary(false),
+          flags(0) {}
 };
 
 /**
  * @brief Table information structure (C++ class)
  */
 class TableInfo {
-public:
+   public:
     oid_t table_id;
     std::string name;
     std::vector<ColumnInfo> columns;
@@ -109,13 +100,7 @@ class TableInfo {
     uint64_t created_at;
     uint64_t modified_at;
 
-    TableInfo()
-        : table_id(0)
-        , num_rows(0)
-        , flags(0)
-        , created_at(0)
-        , modified_at(0)
-    {}
+    TableInfo() : table_id(0), num_rows(0), flags(0), created_at(0), modified_at(0) {}
 
     /**
      * @brief Get column by name
@@ -142,47 +127,39 @@ class TableInfo {
     /**
      * @brief Get number of columns
      */
-    uint16_t num_columns() const {
-        return static_cast<uint16_t>(columns.size());
-    }
+    uint16_t num_columns() const { return static_cast<uint16_t>(columns.size()); }
 
     /**
      * @brief Get number of indexes
      */
-    uint16_t num_indexes() const {
-        return static_cast<uint16_t>(indexes.size());
-    }
+    uint16_t num_indexes() const { return static_cast<uint16_t>(indexes.size()); }
 };
 
 /**
  * @brief Database information structure (C++ class)
  */
 class DatabaseInfo {
-public:
+   public:
     oid_t database_id;
     std::string name;
     uint32_t encoding;
     std::string collation;
     std::vector<oid_t> table_ids;
     uint64_t created_at;
 
-    DatabaseInfo()
-        : database_id(0)
-        , encoding(0)
-        , created_at(0)
-    {}
+    DatabaseInfo() : database_id(0), encoding(0), created_at(0) {}
 };
 
 /**
  * @brief System Catalog class
  */
 class Catalog {
-public:
+   public:
     /**
      * @brief Default constructor
      */
     Catalog() : next_oid_(1) {}
-    
+
     /**
      * @brief Create a new catalog
      */
@@ -202,8 +179,7 @@ class Catalog {
      * @brief Create a new table
      * @return Table OID or 0 on error
      */
-    oid_t create_table(const std::string& table_name, 
-                       std::vector<ColumnInfo> columns);
+    oid_t create_table(const std::string& table_name, std::vector<ColumnInfo> columns);
 
     /**
      * @brief Drop a table
@@ -230,8 +206,8 @@ class Catalog {
      * @return Index OID or 0 on error
      */
     oid_t create_index(const std::string& index_name, oid_t table_id,
-                       std::vector<uint16_t> column_positions,
-                       IndexType index_type, bool is_unique);
+                       std::vector<uint16_t> column_positions, IndexType index_type,
+                       bool is_unique);
 
     /**
      * @brief Drop an index
@@ -266,16 +242,12 @@ class Catalog {
     /**
      * @brief Get database info
      */
-    const DatabaseInfo& get_database() const {
-        return database_;
-    }
+    const DatabaseInfo& get_database() const { return database_; }
 
     /**
      * @brief Set database info
      */
-    void set_database(const DatabaseInfo& db) {
-        database_ = db;
-    }
+    void set_database(const DatabaseInfo& db) { database_ = db; }
 
     /**
      * @brief Print catalog contents
@@ -287,7 +259,7 @@ class Catalog {
      */
     uint64_t get_version() const { return version_; }
 
-private:
+   private:
     std::unordered_map<oid_t, std::unique_ptr<TableInfo>> tables_;
     DatabaseInfo database_;
     oid_t next_oid_;
@@ -296,6 +268,6 @@ class Catalog {
     static uint64_t get_current_time();
 };
 
-} // namespace cloudsql
+}  // namespace cloudsql
 
-#endif // SQL_ENGINE_CATALOG_CATALOG_HPP
+#endif  // SQL_ENGINE_CATALOG_CATALOG_HPP

include/common/config.hpp

@@ -9,21 +9,19 @@
 #include <cstdint>
 #include <string>
 
-namespace cloudsql { namespace config {
+namespace cloudsql {
+namespace config {
 
 /**
  * @brief Run modes for the database engine
  */
-enum class RunMode {
-    Embedded = 0,
-    Distributed = 1
-};
+enum class RunMode { Embedded = 0, Distributed = 1 };
 
 /**
  * @brief Server configuration structure (C++ wrapper)
  */
 class Config {
-public:
+   public:
     static constexpr uint16_t DEFAULT_PORT = 5432;
     static constexpr const char* DEFAULT_DATA_DIR = "./data";
     static constexpr int DEFAULT_MAX_CONNECTIONS = 100;
@@ -71,14 +69,14 @@ class Config {
      */
     void print() const;
 
-private:
+   private:
     /**
      * @brief Trim whitespace from string
      */
     static std::string trim(const std::string& str);
 };
 
-} // namespace config
-} // namespace cloudsql
+}  // namespace config
+}  // namespace cloudsql
 
-#endif // SQL_ENGINE_COMMON_CONFIG_HPP
+#endif  // SQL_ENGINE_COMMON_CONFIG_HPP