AddressIndexer configured for unitysdk#788
AddressIndexer configured for unitysdk#788noescape00 merged 6 commits intostratisproject:release/1.2.0.0from
Conversation
| public const int SyncBuffer = 50; | ||
|
|
||
| // Compaction removes info that is used by unity sdk to get UTXOs. Therefore this feature is disabled. | ||
| public const bool CompactionEnabled = false; |
There was a problem hiding this comment.
I think this should only be false when the Unity API is being used, to preserve the previous behaviour and performance characteristics of the indexer.
There was a problem hiding this comment.
With compaction off behavior is the same, performance is increased, but hard drive usage is increased as well.
With compaction on cirrus test it's around 1.1gb. Currently syncing indexer, will know hard drive impact once it's done.
There was a problem hiding this comment.
I think this should only be false when the Unity API is being used
address indexer is used only for unity api as far as I know.
Also once it was on it can't be turned off because it requires reindexing.
So in case compaction feature is not cosnt bool then data if it was enabled before should be persisted to the database.
I think this can be done in a separate PR once we need addr indexer for smth not unity related.
|
@noescape00 , some tests appear to be failing following this PR. |
This reverts commit af829ca.
* fix 1 * disable compaction * rarer rewind purge loging * Show error if state is incorrect * increase purge performance * upgraded litedb and incresed purging efficency
3 participants
Updated litedb to v5
Increased purging speed
Disabled compacting