refactor: Remove wal_uri from config and remove dummy work_dir parameter

[zhanglei1949]

This pull request simplifies and standardizes how the Write-Ahead Log (WAL) directory is handled throughout the codebase. The main changes remove the wal_uri configuration from NeugDBConfig, eliminate the parse_wal_uri function, and update related interfaces and method signatures to use a direct wal_dir approach. This reduces configuration complexity and potential sources of error related to WAL path resolution.

Configuration Simplification

• Removed the wal_uri field and related logic from NeugDBConfig, so WAL directory handling is now standardized and no longer configurable via template strings. [1] [2] [3]
• Deleted the parse_wal_uri function and all calls to it, removing support for {GRAPH_DATA_DIR} template expansion. [1] [2]

Interface and Method Updates

• Updated constructors and methods in SessionLocalContext, SessionPool, and UpdateTransaction to remove work_dir and wal_uri parameters in favor of directly passing the resolved WAL directory. [1] [2] [3]
• Refactored NeugDB to combine WAL ingestion and graph opening into openGraphAndIngestWals, and updated related method signatures to remove work_dir parameters. [1] [2] [3] [4]

Service and Transaction Logic

• Updated service initialization and WAL ingestion logic to use the new direct WAL directory approach, ensuring consistency across the codebase. [1] [2]

These changes make WAL handling more robust and less error-prone by removing unnecessary configuration indirection.

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.