-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Testing and Documentation #9
Comments
Anthony4m
added a commit
that referenced
this issue
Nov 15, 2024
Anthony4m
added a commit
that referenced
this issue
Dec 1, 2024
Anthony4m
added a commit
that referenced
this issue
Dec 7, 2024
Anthony4m
added a commit
that referenced
this issue
Dec 7, 2024
Anthony4m
added a commit
that referenced
this issue
Dec 7, 2024
Anthony4m
added a commit
that referenced
this issue
Jan 16, 2025
* updated linter configurations * refactored to reduce cyclomatic complexity * refactored to reduce cyclomatic complexity * updated interface to use bytes * updated interface to use bytes * updated interface to use bytes * updated interface to use bytes * updated interface to use bytes * #9 worked on tests * #9 worked on tests
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit corrects inconsistencies in the buffer manager's method names and improves its overall logic. Specifically, it changes `UnPin` to `Unpin`, `GetContents` to `Contents`, and refactors related functionalities for better clarity and consistency. Additionally, it addresses a minor issue with the handling of timeout in tests.
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit improves the buffer manager's logic and clarity by correcting inconsistencies in method names. `UnPin` is changed to `Unpin`, and `GetContents` to `Contents`. Related functionalities are refactored for better consistency. A minor timeout handling issue in tests is also addressed.
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit improves the buffer manager's logic and clarity by correcting inconsistencies in method names. `UnPin` is changed to `Unpin`, and `GetContents` to `Contents`. Related functionalities are refactored for better consistency. A minor timeout handling issue in tests is also addressed.
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit refactors the buffer manager to improve consistency and clarity. Method names like `UnPin` are corrected to `Unpin`, and `GetContents` to `Contents`. Related functionalities are also refactored. A minor timeout issue in tests is fixed.
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit refactors the buffer manager to improve consistency and clarity. Method names like `UnPin` are corrected to `Unpin`, and `GetContents` to `Contents`. Related functionalities are also refactored. A minor timeout issue in tests is fixed. Error messages are improved for better diagnostics. Added locking to prevent race conditions in file operations. Improved error handling and added more comprehensive checks for file permissions and size limits.
Anthony4m
added a commit
that referenced
this issue
Feb 1, 2025
This commit refactors the `LogIterator` to improve clarity, error handling, and consistency. Method names are standardized, and error messages are more informative. Added checks for nil blocks and improved handling of edge cases. The `moveToBlock` function now unpins the previous buffer before pinning a new one. The `Close` method now handles potential errors during unpinning more gracefully.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: