New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GG-37730 Logging improvements for events of WAL segments deletion #2963
base: master
Are you sure you want to change the base?
Conversation
@@ -1067,16 +1067,19 @@ private boolean hasIndex(long absIdx) { | |||
|
|||
int deleted = 0; | |||
|
|||
List<String> deletedSegments = new ArrayList<>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest to use lazy initialization here and in similar places, since this is only for logging
if (cleanedUpSize > 0) | ||
deletedRawSegments.add(desc.idx); | ||
|
||
segmentAware.addSize(desc.idx, -cleanedUpSize); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we move this line under the if
as well?
...ache/ignite/internal/processors/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java
Outdated
Show resolved
Hide resolved
*/ | ||
private Ignite startGrid(Consumer<DataStorageConfiguration> customConfigurator) throws Exception { | ||
private Ignite startGrid(Consumer<DataStorageConfiguration> customConfigurator, IgniteLogger customLog) throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we use Nullable
annotations in GG?
...ache/ignite/internal/processors/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java
Outdated
Show resolved
Hide resolved
...ache/ignite/internal/processors/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java
Outdated
Show resolved
Hide resolved
…rs/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java Co-authored-by: Alexander Polovtcev <alex.polovtcev@gmail.com>
…rs/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java Co-authored-by: Alexander Polovtcev <alex.polovtcev@gmail.com>
…rs/cache/persistence/db/wal/WalDeletionArchiveAbstractTest.java Co-authored-by: Alexander Polovtcev <alex.polovtcev@gmail.com>
…h additional logging (#2963) Co-authored-by: Alexander Polovtcev <alex.polovtcev@gmail.com> Signed-off-by: Alexander Polovtcev <alex.polovtcev@gmail.com>
…h additional logging (#2963) Co-authored-by: Alexander Polovtcev <alex.polovtcev@gmail.com> Signed-off-by: Alexander Polovtcev <alex.polovtcev@gmail.com>
No description provided.