Skip to content
  • Tsvetomir Dimitrov's avatar
    `RollingSessionWindow` cleanup (#7204) · eb1ed63b
    Tsvetomir Dimitrov authored
    
    
    * Replace `RollingSessionWindow` with `RuntimeInfo` - initial commit
    
    * Fix tests in import
    
    * Fix the rest of the tests
    
    * Remove dead code
    
    * Fix todos
    
    * Simplify session caching
    
    * Comments for `SessionInfoProvider`
    
    * Separate `SessionInfoProvider` from `State`
    
    * `cache_session_info_for_head` becomes freestanding function
    
    * Remove unneeded `mut` usage
    
    * fn session_info -> fn get_session_info() to avoid name clashes. The function also tries to initialize `SessionInfoProvider`
    
    * Fix SessionInfo retrieval
    
    * Code cleanup
    
    * Don't wrap `SessionInfoProvider` in an `Option`
    
    * Remove `earliest_session()`
    
    * Remove pre-caching -> wip
    
    * Fix some tests and code cleanup
    
    * Fix all tests
    
    * Fixes in tests
    
    * Fix comments, variable names and small style changes
    
    * Fix a warning
    
    * impl From<SessionWindowSize> for NonZeroUsize
    
    * Fix logging for `get_session_info` - remove redundant logs and decrease log level to DEBUG
    
    * Code review feedback
    
    * Storage migration removing `COL_SESSION_WINDOW_DATA` from parachains db
    
    * Remove `col_session_data` usages
    
    * Storage migration clearing columns w/o removing them
    
    * Remove session data column usages from `approval-voting` and `dispute-coordinator` tests
    
    * Add some test cases from `RollingSessionWindow` to `dispute-coordinator` tests
    
    * Fix formatting in initialized.rs
    
    * Fix a corner case in `SessionInfo` caching for `dispute-coordinator`
    
    * Remove `RollingSessionWindow` ;(
    
    * Revert "Fix formatting in initialized.rs"
    
    This reverts commit 0f94664ec9f3a7e3737a30291195990e1e7065fc.
    
    * v2 to v3 migration drops `COL_DISPUTE_COORDINATOR_DATA` instead of clearing it
    
    * Fix `NUM_COLUMNS` in `approval-voting`
    
    * Use `columns::v3::NUM_COLUMNS` when opening db
    
    * Update node/service/src/parachains_db/upgrade.rs
    
    Co-authored-by: default avatarAndrei Sandu <[email protected]>
    
    * Don't write in `COL_DISPUTE_COORDINATOR_DATA` for `test_rocksdb_migrate_2_to_3`
    
    * Fix `NUM+COLUMNS` in approval_voting
    
    * Fix formatting
    
    * Fix columns usage
    
    * Clarification comments about the different db versions
    
    ---------
    
    Co-authored-by: default avatarAndrei Sandu <[email protected]>
    eb1ed63b