Skip to content

Comments

Geofiles: Revert related changes for now, waiting for better changes#5557

Merged
RPRX merged 2 commits intomainfrom
revert
Jan 17, 2026
Merged

Geofiles: Revert related changes for now, waiting for better changes#5557
RPRX merged 2 commits intomainfrom
revert

Conversation

@Fangliding
Copy link
Member

@Fangliding Fangliding commented Jan 17, 2026

As said

@Fangliding
Revert geofile change
1fc5b7a 
This reverts commit 36425d2.

This reverts commit 961c352.

This reverts commit c715154.
@Fangliding
Revert "Geofiles: Implement mmap in filesystem to reduce ram usage (#…
0fc258e 
…5480)"

This reverts commit 5d94a62.
@RPRX RPRX changed the title Revert Geofiles: Revert related changes for now, waiting for better changes Jan 17, 2026
@RPRX RPRX merged commit b17ca39 into main Jan 17, 2026
78 checks passed
RPRX pushed a commit that referenced this pull request Jan 17, 2026
@Fangliding @RPRX
Geofiles: Revert related changes for now, waiting for better changes (#…
5f74741 
…5557)

Reverts 5d94a62 c715154 961c352 36425d2

Fixes #5538 #5536
@BrewTestBot BrewTestBot mentioned this pull request Jan 18, 2026
Merged
1 task
@Meo597
Copy link
Collaborator

Meo597 commented Jan 20, 2026

我刚才 debug 了一下
我不明白为什么要降峰值内存，反正都会被回收

@Fangliding
Copy link
Member Author

Fangliding commented Jan 20, 2026

因为如果硬限制最大内存 能不能跑就取决于启动这一下峰值

@Meo597
Copy link
Collaborator

Meo597 commented Jan 20, 2026

我 pr 了，感觉应该和他这几个被 revert 掉的效果一样

@BrewTestBot BrewTestBot mentioned this pull request Jan 23, 2026
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Fangliding @Meo597 @RPRX