oknozor/pijul - Change TYAKEAJLABCZQDYAI4YBGIJNQ7HJS4DVULEGPCZOGJPJUYYNR6TAC

A better estimate of the maximum number of open patches we can keep (Unix-only at the moment)

Created by pmeunier on August 17, 2021

TYAKEAJLABCZQDYAI4YBGIJNQ7HJS4DVULEGPCZOGJPJUYYNR6TAC

Dependencies

In channels

main

Change contents

Insertion in pijul/src/repository.rs at line 28 [7.21431]

[6.346]

[7.21994]


#[cfg(unix)]
pub fn max_files() -> usize {
    let n = if let Ok((n, _)) = rlimit::getrlimit(rlimit::Resource::NOFILE) {
        (n as usize / (2 * num_cpus::get())).max(1)
    } else {
        256
    };
    debug!("max_files = {:?}", n);
    n
}
#[cfg(not(unix))]
pub fn max_files() -> usize {
    1
}

Replacement in pijul/src/repository.rs at line 99 [7.21431]

∅:D[8.172] → [7.24102:24200]

∅:D[9.358] → [7.24102:24200]

B:BD[7.24102] → [7.24102:24200]

            changes: libpijul::changestore::filesystem::FileSystem::from_root(&working_copy_dir),

[8.172]

[7.24200]

            changes: libpijul::changestore::filesystem::FileSystem::from_root(
                &working_copy_dir,
                crate::repository::max_files(),
            ),

Replacement in pijul/src/repository.rs at line 136 [7.21431]

∅:D[8.268] → [7.25068:25157]

∅:D[9.503] → [7.25068:25157]

B:BD[7.25068] → [7.25068:25157]

                changes: libpijul::changestore::filesystem::FileSystem::from_root(&cur),

[8.268]

[7.25157]

                changes: libpijul::changestore::filesystem::FileSystem::from_root(
                    &cur,
                    max_files(),
                ),

Replacement in pijul/src/remote/mod.rs at line 789 [7.52474]

B:BD[7.61025] → [7.61025:61122]

                let changes = libpijul::changestore::filesystem::FileSystem::from_root(&l.root);

[7.61025]

[2.21]

                let changes = libpijul::changestore::filesystem::FileSystem::from_root(
                    &l.root,
                    crate::repository::max_files(),
                );

Replacement in pijul/src/remote/mod.rs at line 865 [7.52474]

B:BD[10.2569] → [10.2569:2665]

                let store = libpijul::changestore::filesystem::FileSystem::from_changes(local);

[10.2569]

[5.685]

                let store = libpijul::changestore::filesystem::FileSystem::from_changes(
                    local,
                    crate::repository::max_files(),
                );

Replacement in pijul/src/remote/local.rs at line 62 [7.79745]

∅:D[11.4367] → [7.80806:80896]

B:BD[7.80806] → [7.80806:80896]

        let store = libpijul::changestore::filesystem::FileSystem::from_root(&self.root);

[11.4367]

[7.80896]

        let store = libpijul::changestore::filesystem::FileSystem::from_root(
            &self.root,
            crate::repository::max_files(),
        );

Replacement in pijul/src/remote/local.rs at line 124 [7.79745]

B:BD[7.82268] → [7.82268:82358]

        let store = libpijul::changestore::filesystem::FileSystem::from_root(&self.root);

[7.82268]

[8.1180]

        let store = libpijul::changestore::filesystem::FileSystem::from_root(
            &self.root,
            crate::repository::max_files(),
        );

Insertion in pijul/Cargo.toml at line 92 [7.196374]
[3.1493]
[12.309]
```
rlimit = "0.6"
```
Replacement in libpijul/src/changestore/filesystem.rs at line 4 [7.824013]
B:BD[7.824179] → [4.104729:104753]
```
use parking_lot::Mutex;
```
[7.824179]
[7.824179]
```
use std::cell::RefCell;
```
Deletion in libpijul/src/changestore/filesystem.rs at line 6 [7.824013]
B:BD[7.824211] → [4.104754:104774]
```
use std::sync::Arc;
```
Deletion in libpijul/src/changestore/filesystem.rs at line 7 [7.824013]
B:BD[7.824253] → [7.824253:824292]
```
const CHANGE_CACHE_SIZE: usize = 100;
```

Replacement in libpijul/src/changestore/filesystem.rs at line 9 [7.824013]

B:BD[9.87463] → [4.104775:104857]

    change_cache: Arc<Mutex<lru_cache::LruCache<ChangeId, ChangeFile<'static>>>>,

[9.87463]

[7.824493]

    change_cache: RefCell<lru_cache::LruCache<ChangeId, ChangeFile<'static>>>,

Insertion in libpijul/src/changestore/filesystem.rs at line 15 [7.824013]
[9.87604]
[9.87604]
```
        let len = self.change_cache.borrow().capacity();
```

Replacement in libpijul/src/changestore/filesystem.rs at line 18 [7.824013]

B:BD[9.87676] → [4.104858:104951]

            change_cache: Arc::new(Mutex::new(lru_cache::LruCache::new(CHANGE_CACHE_SIZE))),

[9.87676]

[9.87761]

            change_cache: RefCell::new(lru_cache::LruCache::new(len)),

Replacement in libpijul/src/changestore/filesystem.rs at line 61 [7.824013]

B:BD[7.825268] → [7.825268:825324]

    pub fn from_root<P: AsRef<Path>>(root: P) -> Self {

[7.825268]

[7.825324]

    pub fn from_root<P: AsRef<Path>>(root: P, cap: usize) -> Self {

Replacement in libpijul/src/changestore/filesystem.rs at line 64 [7.824013]
B:BD[7.825437] → [7.825437:825477]
```
        Self::from_changes(changes_dir)
```
[7.825437]
[7.825477]
```
        Self::from_changes(changes_dir, cap)
```

Replacement in libpijul/src/changestore/filesystem.rs at line 69 [7.824013]

B:BD[7.825612] → [7.825612:825668]

    pub fn from_changes(changes_dir: PathBuf) -> Self {

[7.825612]

[7.825668]

    pub fn from_changes(changes_dir: PathBuf, cap: usize) -> Self {

Replacement in libpijul/src/changestore/filesystem.rs at line 73 [7.824013]

B:BD[7.825791] → [4.104952:105045]

            change_cache: Arc::new(Mutex::new(lru_cache::LruCache::new(CHANGE_CACHE_SIZE))),

[7.825791]

[9.87935]

            change_cache: RefCell::new(lru_cache::LruCache::new(cap)),

Replacement in libpijul/src/changestore/filesystem.rs at line 82 [7.824013]

B:BD[7.826014] → [4.105046:105186]

        parking_lot::MutexGuard<
            lru_cache::LruCache<crate::pristine::ChangeId, crate::change::ChangeFile<'static>>,
        >,

[7.826014]

[13.35450]

        std::cell::RefMut<lru_cache::LruCache<ChangeId, ChangeFile<'static>>>,

Replacement in libpijul/src/changestore/filesystem.rs at line 85 [7.824013]

B:BD[7.826129] → [4.105187:105244]

        let mut change_cache = self.change_cache.lock();

[7.826129]

[9.88078]

        let mut change_cache = self.change_cache.borrow_mut();

Replacement in libpijul/src/changestore/filesystem.rs at line 122 [7.824013]

B:BD[7.827293] → [4.105245:105301]

            self.change_cache.lock().remove(change_id);

[7.827293]

[7.827395]

            self.change_cache.borrow_mut().remove(change_id);

Replacement in libpijul/src/changestore/filesystem.rs at line 132 [7.824013]

B:BD[7.827590] → [4.105302:105377]

            if let Some(l) = self.change_cache.lock().get_mut(change_id) {

[7.827590]

[9.88498]

            if let Some(l) = self.change_cache.borrow_mut().get_mut(change_id) {

Insertion in Cargo.lock at line 1512 [7.1030385]
[14.33884]
[14.33884]
```
 "rlimit",
```

Insertion in Cargo.lock at line 1776 [7.1030385]

[14.41056]

]
[[package]]
name = "rlimit"
version = "0.6.2"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "cc0bf25554376fd362f54332b8410a625c71f15445bca32ffdfdf4ec9ac91726"
dependencies = [
 "libc",