hanahmily opened a new pull request, #1028: URL: https://github.com/apache/skywalking-banyandb/pull/1028
### Organize access logs under a dedicated subdirectory This feature improves log organization by placing all access logs under a dedicated `accesslog` subdirectory within the configured access log root path. #### Changes: - Appends `/accesslog` to the access log root path during server startup - Creates the subdirectory automatically if it does not exist - All ingestion and query access logs are written to the subdirectory - Uses `pkg/fs` for filesystem operations with proper logging - Uses project standard `storage.DirPerm` constant for directory permissions #### Checklist: - [x] Feature description included above - [x] [`CHANGES` log](https://github.com/apache/skywalking-banyandb/blob/main/CHANGES.md) updated (added to 0.11.0 section) - [x] Implementation follows project conventions (pkg/fs, storage.DirPerm constant) - [x] No breaking changes - feature is transparent to all services #### Test Plan: 1. Start BanyanDB with access logging enabled: ```bash banyand-server standalone \ --enable-ingestion-access-log \ --enable-query-access-log \ --access-log-root-path=/logs ``` 2. Verify that logs are created under `/logs/accesslog/` directory 3. Verify that existing access log tests pass 4. Generate some traffic to ensure logs are being written to the subdirectory #### Impact: - Minimal impact - single point of modification in server.go - No changes required to service implementations - Backward compatible - only changes the log file location -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
