github-actions[bot] commented on code in PR #42507: URL: https://github.com/apache/doris/pull/42507#discussion_r1817773962
########## be/src/util/jvm_metrics.cpp: ########## @@ -182,193 +188,260 @@ void JvmMetrics::update() { } } -void JvmStats::init(JNIEnv* ENV) { - env = ENV; - _managementFactoryClass = env->FindClass("java/lang/management/ManagementFactory"); - if (_managementFactoryClass == nullptr) { - LOG(WARNING) - << "Class java/lang/management/ManagementFactory Not Find.JVM monitoring fails."; - return; - } +Status JvmStats::init() { Review Comment: warning: function 'init' has cognitive complexity of 147 (threshold 50) [readability-function-cognitive-complexity] ```cpp Status JvmStats::init() { ^ ``` <details> <summary>Additional context</summary> **be/src/util/jvm_metrics.cpp:192:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetJNIEnv(&env)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:192:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetJNIEnv(&env)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:194:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/ManagementFactory", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:194:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/ManagementFactory", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:197:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryMXBeanMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:197:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryMXBeanMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:201:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryUsage", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:201:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryUsage", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:204:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageUsedMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:204:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageUsedMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:207:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageCommittedMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:207:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageCommittedMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:210:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageMaxMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:210:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryUsageMaxMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:213:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryMXBean", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:213:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryMXBean", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:216:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getHeapMemoryUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:216:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getHeapMemoryUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:219:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getNonHeapMemoryUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:219:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getNonHeapMemoryUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:223:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:223:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:228:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/util/List", &_listClass)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:228:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/util/List", &_listClass)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:230:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getListSizeMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:230:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getListSizeMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:233:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getListUseIndexMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:233:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getListUseIndexMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:236:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryPoolMXBean", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:236:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/MemoryPoolMXBean", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:239:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryPoolMXBeanUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:239:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryPoolMXBeanUsageMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:243:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryPollMXBeanPeakMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:243:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getMemoryPollMXBeanPeakMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:246:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:246:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:250:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getThreadMXBeanMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:250:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, _getThreadMXBeanMethod, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:254:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:254:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:259:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/GarbageCollectorMXBean", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:259:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/GarbageCollectorMXBean", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:262:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:262:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:266:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:266:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:270:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:270:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:274:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/ThreadMXBean", ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:274:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/management/ThreadMXBean", ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:277:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:277:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:282:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:282:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:288:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:288:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:293:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:293:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:296:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:296:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:302:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/Thread$State", &_threadStateClass)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:302:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::GetGlobalClassRef(env, "java/lang/Thread$State", &_threadStateClass)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:304:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:304:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:308:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:308:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:312:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:312:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:315:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:315:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:319:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:319:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:322:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:322:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:326:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, newThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:326:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, newThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:328:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, newThreadStateObj, &_newThreadStateObj)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:328:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, newThreadStateObj, &_newThreadStateObj)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:330:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, runnableThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:330:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, runnableThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:332:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:332:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:335:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, blockedThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:335:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, blockedThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:337:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, blockedThreadStateObj, &_blockedThreadStateObj)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:337:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, blockedThreadStateObj, &_blockedThreadStateObj)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:339:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, waitingThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:339:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION(jobject, waitingThreadStateObj, env, ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:341:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, waitingThreadStateObj, &_waitingThreadStateObj)); ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:341:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, waitingThreadStateObj, &_waitingThreadStateObj)); ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:343:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:343:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:346:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, timedWaitingThreadStateObj, ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:346:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR(JniUtil::LocalToGlobalRef(env, timedWaitingThreadStateObj, ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` **be/src/util/jvm_metrics.cpp:349:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:42:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:349:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp JNI_CALL_METHOD_CHECK_EXCEPTION( ^ ``` **be/src/util/jni-util.h:54:** expanded from macro 'JNI_CALL_METHOD_CHECK_EXCEPTION' ```cpp RETURN_ERROR_IF_EXC(env) ^ ``` **be/src/util/jni-util.h:43:** expanded from macro 'RETURN_ERROR_IF_EXC' ```cpp if (env->ExceptionCheck()) [[unlikely]] \ ^ ``` **be/src/util/jvm_metrics.cpp:352:** +1, including nesting penalty of 0, nesting level increased to 1 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR' ```cpp do { \ ^ ``` **be/src/util/jvm_metrics.cpp:352:** +2, including nesting penalty of 1, nesting level increased to 2 ```cpp RETURN_IF_ERROR( ^ ``` **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR' ```cpp if (UNLIKELY(!_status_.ok())) { \ ^ ``` </details> ########## be/src/util/jvm_metrics.cpp: ########## @@ -182,193 +188,260 @@ } } -void JvmStats::init(JNIEnv* ENV) { - env = ENV; - _managementFactoryClass = env->FindClass("java/lang/management/ManagementFactory"); - if (_managementFactoryClass == nullptr) { - LOG(WARNING) - << "Class java/lang/management/ManagementFactory Not Find.JVM monitoring fails."; - return; - } +Status JvmStats::init() { Review Comment: warning: function 'init' exceeds recommended size/complexity thresholds [readability-function-size] ```cpp Status JvmStats::init() { ^ ``` <details> <summary>Additional context</summary> **be/src/util/jvm_metrics.cpp:190:** 169 lines including whitespace and comments (threshold 80) ```cpp Status JvmStats::init() { ^ ``` </details> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org