Ramananeesh/issue15 switch atomic counters to fast path metadata instead of prev

[ramananeesh]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a new atomic counter approach for the ConcurrentQuITBTree implementation by creating a second atomic version (ConcurrentQuITBTreeAtomic2) that switches from using previous node metadata to fast path metadata for atomic operations. The implementation aims to improve concurrency performance by refining how fast path metadata is managed atomically.

• Created ConcurrentQuITBTreeAtomic2 as a new atomic tree implementation variant
• Refactored atomic counter management to use fast path metadata instead of previous node metadata
• Updated build configuration and analysis tools to support the new tree variant

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/tree_analysis.cpp	Added support for the new FOR_CONCURRENT_QUIT_ATOMIC2 preprocessor definition
src/include/trees/ConcurrentQuITBTreeAtomic2.hpp	New complete implementation of the atomic2 variant with fast path metadata approach
src/include/trees/ConcurrentQuITBTreeAtomic.hpp	Modified to move fp_sorted from standalone atomic to part of fast_path_metadata struct
src/include/trees.hpp	Added include for the new ConcurrentQuITBTreeAtomic2 header
src/CMakeLists.txt	Added build target for concurrent-quit-atomic2 tree type
src/include/utils/executor.hpp	Changed debug output from logging to cerr for missing keys
results/analysis/analysis.ipynb	Updated analysis filter to focus on concurrent_quit_appends variant
config.toml	Changed NUM_THREADS from 16 to 2
CMakeLists.txt	Added DEBUG flag to debug build configuration
.vscode/settings.json	Added comprehensive C++ file associations for IDE support
.vscode/launch.json	Added debug launch configuration for the new atomic2 variant

Comments suppressed due to low confidence (1)

src/include/trees/ConcurrentQuITBTreeAtomic2.hpp:55

• The concurrent flag is set to false for a class named 'ConcurrentQuITBTreeAtomic2', which is misleading. This should likely be true or the naming should be reconsidered.

    static constexpr const bool concurrent = false;