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
IGNITE-12505 Improve logging of data regions on startup. #7209
Conversation
@@ -18,11 +18,15 @@ | |||
package org.apache.ignite.internal.processors.cache.persistence; | |||
|
|||
import java.io.File; | |||
import java.text.DecimalFormat; |
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 really need visual representation stuff in IgniteCacheDatabaseSharedManager
? Can we extract it somewhere?
@@ -99,6 +106,9 @@ | |||
/** DataRegionConfiguration name reserved for internal caches. */ | |||
public static final String SYSTEM_DATA_REGION_NAME = "sysMemPlc"; | |||
|
|||
public static Set<String> INTERNAL_DATA_REGION_NAMES = Collections.unmodifiableSet( |
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.
Missing javadoc.
U.quiet(false, "Data Regions Started: " + dataRegionMap.size()); | ||
|
||
U.quietMultipleLines(false, IgniteKernal.dataStorageReport(this, dblFmt, false)); | ||
} else if (log.isInfoEnabled()) { |
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.
According to code style else if
should come from a new line.
assertTrue(msg, fullLog.matches("(?s).*Outbound messages queue \\[size=.*].*")); | ||
assertTrue(msg, fullLog.matches("(?s).*Public thread pool \\[active=.*, idle=.*, qSize=.*].*")); | ||
assertTrue(msg, fullLog.matches("(?s).*System thread pool \\[active=.*, idle=.*, qSize=.*].*")); | ||
assertTrue(msg, fullLog.matches("(?s).*" + CUSTOM_EXECUTOR_0 + " \\[active=.*, idle=.*, qSize=.*].*")); | ||
assertTrue(msg, fullLog.matches("(?s).*" + CUSTOM_EXECUTOR_1 + " \\[active=.*, idle=.*, qSize=.*].*")); | ||
} | ||
|
||
|
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.
Extra empty line (according to code style).
double freeOffHeapPct = offHeapMaxSummary > 0 ? | ||
((double)((offHeapMaxSummary - offHeapUsedSummary) * 100)) / offHeapMaxSummary : -1; | ||
|
||
return (includeMemoryStatistics ? |
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.
It is hard for me to read ternary operator for this statement. I suggest to extract methods assembling message pieces.
No description provided.