Skip to content

Commit 996abce

Browse files
committed
Rename Non-native to Canonical
1 parent 8013d4f commit 996abce

File tree

14 files changed

+64
-64
lines changed

14 files changed

+64
-64
lines changed

ouroboros-consensus-cardano/src/unstable-cardano-tools/Cardano/Tools/DBAnalyser/Run.hs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,9 @@ openLedgerDB args = do
9999
(LedgerDBSnapshotEvent >$< LedgerDB.lgrTracer args)
100100
(LedgerDB.lgrHasFS args)
101101
( flip
102-
LedgerDB.V2.NonNativeSnapshotsFS
102+
LedgerDB.V2.CanonicalSnapshotsFS
103103
(LedgerDB.lgrHasFS args)
104-
<$> LedgerDB.lgrNonNativeSnapshotsFS args
104+
<$> LedgerDB.lgrCanonicalSnapshotsFS args
105105
)
106106
let initDb = LedgerDB.V2.mkInitDb args (\_ -> pure (error "no stream")) snapManager res
107107
LedgerDB.openDBInternal args initDb snapManager emptyStream genesisPoint

ouroboros-consensus-diffusion/src/ouroboros-consensus-diffusion/Ouroboros/Consensus/Node.hs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,7 @@ data
359359
, srnChainDbValidateOverride :: Bool
360360
-- ^ If @True@, validate the ChainDB on init no matter what
361361
, srnDatabasePath :: NodeDatabasePaths
362-
, srnNonNativeSnapshotsPath :: Maybe FilePath
362+
, srnCanonicalSnapshotsPath :: Maybe FilePath
363363
-- ^ Location of the DBs
364364
, srnDiffusionArguments :: Cardano.Diffusion.CardanoNodeArguments m
365365
, srnDiffusionConfiguration :: Cardano.Diffusion.CardanoConfiguration m
@@ -1065,8 +1065,8 @@ stdLowLevelRunNodeArgsIO
10651065
. ChainDB.updateTracer srnTraceChainDB
10661066
. maybe
10671067
id
1068-
(\fp -> ChainDB.enableNonNativeSnapshots (SomeHasFS $ ioHasFS $ MountPoint fp))
1069-
srnNonNativeSnapshotsPath
1068+
(\fp -> ChainDB.enableCanonicalSnapshots (SomeHasFS $ ioHasFS $ MountPoint fp))
1069+
srnCanonicalSnapshotsPath
10701070
. ( if not srnChainDbValidateOverride
10711071
then id
10721072
else ChainDB.ensureValidateAll

ouroboros-consensus/src/ouroboros-consensus-lsm/Ouroboros/Consensus/Storage/LedgerDB/V2/LSM.hs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -233,13 +233,13 @@ snapshotManager ::
233233
CodecConfig blk ->
234234
Tracer m (TraceSnapshotEvent blk) ->
235235
SomeHasFS m ->
236-
Maybe (NonNativeSnapshotsFS m) ->
236+
Maybe (CanonicalSnapshotsFS m) ->
237237
SnapshotManager m m blk (StateRef m (ExtLedgerState blk))
238-
snapshotManager session ccfg tracer fs mNonNative =
238+
snapshotManager session ccfg tracer fs mCanonical =
239239
SnapshotManager
240240
{ listSnapshots = defaultListSnapshots fs
241241
, deleteSnapshot = implDeleteSnapshot session fs tracer
242-
, takeSnapshot = implTakeSnapshot ccfg tracer fs mNonNative
242+
, takeSnapshot = implTakeSnapshot ccfg tracer fs mCanonical
243243
}
244244

245245
newLSMLedgerTablesHandle ::
@@ -360,11 +360,11 @@ implTakeSnapshot ::
360360
CodecConfig blk ->
361361
Tracer m (TraceSnapshotEvent blk) ->
362362
SomeHasFS m ->
363-
Maybe (NonNativeSnapshotsFS m) ->
363+
Maybe (CanonicalSnapshotsFS m) ->
364364
Maybe String ->
365365
StateRef m (ExtLedgerState blk) ->
366366
m (Maybe (DiskSnapshot, RealPoint blk))
367-
implTakeSnapshot ccfg tracer shfs mNonNativeFS suffix st =
367+
implTakeSnapshot ccfg tracer shfs mCanonicalFS suffix st =
368368
case pointToWithOriginRealPoint (castPoint (getTip $ state st)) of
369369
Origin -> return Nothing
370370
NotOrigin t -> do
@@ -378,15 +378,15 @@ implTakeSnapshot ccfg tracer shfs mNonNativeFS suffix st =
378378
stateCRC <-
379379
encloseTimedWith (TookSnapshot snapshot t >$< tracer) $
380380
writeSnapshot shfs (encodeDiskExtLedgerState ccfg) snapshot st
381-
takeNonNativeSnapshot
381+
takeCanonicalSnapshot
382382
(($ t) >$< tracer)
383383
snapshot
384384
(duplicate (tables st))
385385
close
386386
(\hdl -> yield (Proxy @LSM) (YieldLSM 1000 hdl) (state st))
387387
(state st)
388388
stateCRC
389-
mNonNativeFS
389+
mCanonicalFS
390390

391391
return $ Just (snapshot, t)
392392

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/ChainDB/Impl/Args.hs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ module Ouroboros.Consensus.Storage.ChainDB.Impl.Args
1313
, completeChainDbArgs
1414
, defaultArgs
1515
, enableLedgerEvents
16-
, enableNonNativeSnapshots
16+
, enableCanonicalSnapshots
1717
, ensureValidateAll
1818
, updateQueryBatchSize
1919
, updateSnapshotPolicyArgs
@@ -264,9 +264,9 @@ enableLedgerEvents args =
264264
}
265265
}
266266

267-
enableNonNativeSnapshots :: SomeHasFS m -> ChainDbArgs f m blk -> ChainDbArgs f m blk
268-
enableNonNativeSnapshots shfs args =
269-
args{cdbLgrDbArgs = (cdbLgrDbArgs args){LedgerDB.lgrNonNativeSnapshotsFS = Just shfs}}
267+
enableCanonicalSnapshots :: SomeHasFS m -> ChainDbArgs f m blk -> ChainDbArgs f m blk
268+
enableCanonicalSnapshots shfs args =
269+
args{cdbLgrDbArgs = (cdbLgrDbArgs args){LedgerDB.lgrCanonicalSnapshotsFS = Just shfs}}
270270

271271
{-------------------------------------------------------------------------------
272272
Relative mount points

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ openDB
9494
(configCodec . getExtLedgerCfg . ledgerDbCfg $ lgrConfig args)
9595
(LedgerDBSnapshotEvent >$< lgrTracer args)
9696
(lgrHasFS args)
97-
(flip NonNativeSnapshotsFS (lgrHasFS args) <$> lgrNonNativeSnapshotsFS args)
97+
(flip CanonicalSnapshotsFS (lgrHasFS args) <$> lgrCanonicalSnapshotsFS args)
9898
let initDb = V2.mkInitDb args getBlock snapManager res
9999
doOpenDB args initDb snapManager stream replayGoal
100100

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB/Args.hs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ data LedgerDbArgs f m blk = LedgerDbArgs
4848
, lgrTracer :: Tracer m (TraceEvent blk)
4949
, lgrBackendArgs :: LedgerDbBackendArgs m blk
5050
, lgrRegistry :: HKD f (ResourceRegistry m)
51-
, lgrNonNativeSnapshotsFS :: Maybe (SomeHasFS m)
52-
-- ^ If Just, enable non-native snapshots.
51+
, lgrCanonicalSnapshotsFS :: Maybe (SomeHasFS m)
52+
-- ^ If Just, enable canonical snapshots.
5353
, lgrQueryBatchSize :: QueryBatchSize
5454
, lgrStartSnapshot :: Maybe DiskSnapshot
5555
-- ^ If provided, the ledgerdb will start using said snapshot and fallback
@@ -75,7 +75,7 @@ defaultArgs backendArgs =
7575
lgrBackendArgs = LedgerDbBackendArgsV2 backendArgs
7676
, lgrRegistry = NoDefault
7777
, lgrStartSnapshot = Nothing
78-
, lgrNonNativeSnapshotsFS = Nothing
78+
, lgrCanonicalSnapshotsFS = Nothing
7979
}
8080

8181
data LedgerDbBackendArgs m blk

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB/Snapshots.hs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -607,8 +607,8 @@ data TraceSnapshotEvent blk
607607
InvalidSnapshot DiskSnapshot (SnapshotFailure blk)
608608
| -- | A snapshot was written to disk.
609609
TookSnapshot DiskSnapshot (RealPoint blk) EnclosingTimed
610-
| -- | A non-native snapshot was written to disk.
611-
TookNonNativeSnapshot DiskSnapshot (RealPoint blk) EnclosingTimed
610+
| -- | A canonical snapshot was written to disk.
611+
TookCanonicalSnapshot DiskSnapshot (RealPoint blk) EnclosingTimed
612612
| -- | An old or invalid on-disk snapshot was deleted
613613
DeletedSnapshot DiskSnapshot
614614
deriving (Generic, Eq, Show)

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB/V1/BackingStore/Impl/InMemory.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -374,4 +374,4 @@ instance StreamingBackend m Mem l where
374374
sink _ (SinkArgs x) = absurd x
375375

376376
instance StreamingBackendV1 m Mem l where
377-
yieldV1 _ _ = error "We do not support streaming non-native snapshots from a V1 InMemory backend"
377+
yieldV1 _ _ = error "We do not support streaming canonical snapshots from a V1 InMemory backend"

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB/V1/Snapshots.hs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ import Ouroboros.Consensus.Storage.LedgerDB.V1.BackingStore
160160
import qualified Ouroboros.Consensus.Storage.LedgerDB.V1.BackingStore as V1
161161
import Ouroboros.Consensus.Storage.LedgerDB.V1.DbChangelog
162162
import Ouroboros.Consensus.Storage.LedgerDB.V1.Lock
163-
import Ouroboros.Consensus.Storage.LedgerDB.V2.Backend (NonNativeSnapshotsFS (..))
163+
import Ouroboros.Consensus.Storage.LedgerDB.V2.Backend (CanonicalSnapshotsFS (..))
164164
import Ouroboros.Consensus.Storage.LedgerDB.V2.InMemory hiding (Args, snapshotManager)
165165
import Ouroboros.Consensus.Util.Args (Complete)
166166
import Ouroboros.Consensus.Util.Enclose
@@ -182,7 +182,7 @@ snapshotManager args p =
182182
(configCodec . getExtLedgerCfg . ledgerDbCfg $ lgrConfig args)
183183
(LedgerDBSnapshotEvent >$< lgrTracer args)
184184
(SnapshotsFS (lgrHasFS args))
185-
(flip NonNativeSnapshotsFS (lgrHasFS args) <$> lgrNonNativeSnapshotsFS args)
185+
(flip CanonicalSnapshotsFS (lgrHasFS args) <$> lgrCanonicalSnapshotsFS args)
186186

187187
snapshotManager' ::
188188
( IOLike m
@@ -193,7 +193,7 @@ snapshotManager' ::
193193
CodecConfig blk ->
194194
Tracer m (TraceSnapshotEvent blk) ->
195195
SnapshotsFS m ->
196-
Maybe (NonNativeSnapshotsFS m) ->
196+
Maybe (CanonicalSnapshotsFS m) ->
197197
SnapshotManager m (ReadLocked m) blk (StrictTVar m (DbChangelog' blk), BackingStore' m blk)
198198
snapshotManager' p ccfg tracer sfs@(SnapshotsFS fs) mNNFS =
199199
SnapshotManager
@@ -232,12 +232,12 @@ implTakeSnapshot ::
232232
CodecConfig blk ->
233233
Tracer m (TraceSnapshotEvent blk) ->
234234
SnapshotsFS m ->
235-
Maybe (NonNativeSnapshotsFS m) ->
235+
Maybe (CanonicalSnapshotsFS m) ->
236236
BackingStore' m blk ->
237237
-- | Override for snapshot numbering
238238
Maybe String ->
239239
ReadLocked m (Maybe (DiskSnapshot, RealPoint blk))
240-
implTakeSnapshot (V1.V1Args _ (V1.SomeBackendArgs (_ :: V1.Args m backend))) ldbvar ccfg tracer (SnapshotsFS hasFS) mNonNativeFS backingStore suffix = readLocked $ do
240+
implTakeSnapshot (V1.V1Args _ (V1.SomeBackendArgs (_ :: V1.Args m backend))) ldbvar ccfg tracer (SnapshotsFS hasFS) mCanonicalFS backingStore suffix = readLocked $ do
241241
state <- changelogLastFlushedState <$> readTVarIO ldbvar
242242
case pointToWithOriginRealPoint (castPoint (getTip state)) of
243243
Origin ->
@@ -253,15 +253,15 @@ implTakeSnapshot (V1.V1Args _ (V1.SomeBackendArgs (_ :: V1.Args m backend))) ldb
253253
stateCRC <-
254254
encloseTimedWith (TookSnapshot snapshot t >$< tracer) $
255255
writeSnapshot hasFS backingStore (encodeDiskExtLedgerState ccfg) snapshot state
256-
takeNonNativeSnapshot
256+
takeCanonicalSnapshot
257257
(($ t) >$< tracer)
258258
snapshot
259259
(bsValueHandle backingStore)
260260
bsvhClose
261261
(\vh -> yieldV1 (Proxy @backend) vh state)
262262
state
263263
stateCRC
264-
mNonNativeFS
264+
mCanonicalFS
265265
return $ Just (snapshot, t)
266266

267267
-- | Write snapshot to disk

ouroboros-consensus/src/ouroboros-consensus/Ouroboros/Consensus/Storage/LedgerDB/V2/Backend.hs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
module Ouroboros.Consensus.Storage.LedgerDB.V2.Backend
1010
( -- * Backend API
1111
Backend (..)
12-
, NonNativeSnapshotsFS (..)
12+
, CanonicalSnapshotsFS (..)
1313

1414
-- * Existentials
1515
, SomeBackendTrace (..)
@@ -83,13 +83,13 @@ class NoThunks (Resources m backend) => Backend m backend blk where
8383
CodecConfig blk ->
8484
Tracer m (TraceSnapshotEvent blk) ->
8585
SomeHasFS m ->
86-
Maybe (NonNativeSnapshotsFS m) ->
86+
Maybe (CanonicalSnapshotsFS m) ->
8787
SnapshotManager m m blk (StateRef m (ExtLedgerState blk))
8888

89-
-- | Arguments required if non-native snapshots are enabled.
90-
data NonNativeSnapshotsFS m = NonNativeSnapshotsFS
91-
{ nnNonNativeHasFS :: SomeHasFS m
92-
-- ^ The FS on which non-native snapshots are stored
89+
-- | Arguments required if canonical snapshots are enabled.
90+
data CanonicalSnapshotsFS m = CanonicalSnapshotsFS
91+
{ nnCanonicalHasFS :: SomeHasFS m
92+
-- ^ The FS on which canonical snapshots are stored
9393
, nnNativeHasFS :: SomeHasFS m
9494
-- ^ The FS on which native snapshots are stored
9595
}

0 commit comments

Comments
 (0)