-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chore(cairo): virtual memory refactoring #1208
Conversation
bluestreak01
commented
Jul 20, 2021
- new contiguous read-write memory class
- bitmap index writer uses contiguous memory
core/src/main/java/io/questdb/cairo/vm/ContiguousMappedReadOnlyMemory.java
Outdated
Show resolved
Hide resolved
…ve dirty pages under any circumstances
…, fixed size bug in ContinuousVirtualMemory
refactored cache key
…oncrete implementations
…ng) file handles intermittently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd name interfaced / clases in reverse patter:
MemoryXYZ
also ReadMemory, WriteMemory to reduce to R
MemoryR
, MemoryW
This way when you type Memory|
you're autosuggested all R,W,A,C,P,M combinations
Also I'd move creation of this to single factory class to have single interface creating memories, say
class Memory {
static MemorRWC createRWC() {...}
...
}
and put the description of what is the meaning of R,W,..C letter in the Memory class comments.
All this is minor, nice to have imho things
[PR Coverage check]😍 pass : 1202 / 1323 (90.85%) file detail
|