Skip to content

Commit

Permalink
Treat file descriptors numbers as plain longs instead of bytes during…
Browse files Browse the repository at this point in the history
… diagnostic logging.

 'Max number of file descriptors' and 'Number of open file descriptors' parameters was treated as bytes instead of numbers.
  • Loading branch information
MishaDemianenko committed May 23, 2016
1 parent 7a97edd commit 3404a28
Showing 1 changed file with 16 additions and 8 deletions.
Expand Up @@ -59,11 +59,11 @@ enum SystemDiagnostics implements DiagnosticsProvider
@Override @Override
void dump( Logger logger ) void dump( Logger logger )
{ {
logOsBeanValue( logger, "Total Physical memory: ", OsBeanUtil.getTotalPhysicalMemory() ); logBytes( logger, "Total Physical memory: ", OsBeanUtil.getTotalPhysicalMemory() );
logOsBeanValue( logger, "Free Physical memory: ", OsBeanUtil.getFreePhysicalMemory() ); logBytes( logger, "Free Physical memory: ", OsBeanUtil.getFreePhysicalMemory() );
logOsBeanValue( logger, "Committed virtual memory: ", OsBeanUtil.getCommittedVirtualMemory() ); logBytes( logger, "Committed virtual memory: ", OsBeanUtil.getCommittedVirtualMemory() );
logOsBeanValue( logger, "Total swap space: ", OsBeanUtil.getTotalSwapSpace() ); logBytes( logger, "Total swap space: ", OsBeanUtil.getTotalSwapSpace() );
logOsBeanValue( logger, "Free swap space: ", OsBeanUtil.getFreeSwapSpace() ); logBytes( logger, "Free swap space: ", OsBeanUtil.getFreeSwapSpace() );
} }
}, },
JAVA_MEMORY( "JVM memory information:" ) JAVA_MEMORY( "JVM memory information:" )
Expand Down Expand Up @@ -97,8 +97,8 @@ void dump( Logger logger )
RuntimeMXBean runtime = ManagementFactory.getRuntimeMXBean(); RuntimeMXBean runtime = ManagementFactory.getRuntimeMXBean();
logger.log( String.format( "Operating System: %s; version: %s; arch: %s; cpus: %s", os.getName(), logger.log( String.format( "Operating System: %s; version: %s; arch: %s; cpus: %s", os.getName(),
os.getVersion(), os.getArch(), os.getAvailableProcessors() ) ); os.getVersion(), os.getArch(), os.getAvailableProcessors() ) );
logOsBeanValue( logger, "Max number of file descriptors: ", OsBeanUtil.getMaxFileDescriptors() ); logLong( logger, "Max number of file descriptors: ", OsBeanUtil.getMaxFileDescriptors() );
logOsBeanValue( logger, "Number of open file descriptors: ", OsBeanUtil.getOpenFileDescriptors() ); logLong( logger, "Number of open file descriptors: ", OsBeanUtil.getOpenFileDescriptors() );
logger.log( "Process id: " + runtime.getName() ); logger.log( "Process id: " + runtime.getName() );
logger.log( "Byte order: " + ByteOrder.nativeOrder() ); logger.log( "Byte order: " + ByteOrder.nativeOrder() );
logger.log( "Local timezone: " + getLocalTimeZone() ); logger.log( "Local timezone: " + getLocalTimeZone() );
Expand Down Expand Up @@ -339,11 +339,19 @@ private static String canonicalize( String path )
} }
} }


private static void logOsBeanValue( Logger logger, String message, long value ) private static void logBytes( Logger logger, String message, long value )
{ {
if ( value != OsBeanUtil.VALUE_UNAVAILABLE ) if ( value != OsBeanUtil.VALUE_UNAVAILABLE )
{ {
logger.log( message + bytes( value ) ); logger.log( message + bytes( value ) );
} }
} }

private static void logLong( Logger logger, String message, long value )
{
if ( value != OsBeanUtil.VALUE_UNAVAILABLE )
{
logger.log( message + value );
}
}
} }

0 comments on commit 3404a28

Please sign in to comment.