Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-5322

Potential crash in Frontend & Catalog JNI startup

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: Impala 2.8.0
    • Fix Version/s: Impala 2.9.0
    • Component/s: Backend
    • Labels:
      None
    • Epic Color:
      ghx-label-5

      Description

       cfg.__set_load_auth_to_local_rules(FLAGS_load_auth_to_local_rules);
        cfg.__set_principal(FLAGS_principal);
        cfg.__set_impala_log_lvl(FlagToTLogLevel(FLAGS_v));
        cfg.__set_non_impala_java_vlog(FlagToTLogLevel(FLAGS_non_impala_java_vlog));
        cfg.__set_inc_stats_size_limit_bytes(FLAGS_inc_stats_size_limit_bytes);
        cfg.__set_lineage_event_log_dir(FLAGS_lineage_event_log_dir);
        cfg.__set_local_library_path(FLAGS_local_library_dir);
        cfg.__set_kudu_operation_timeout_ms(FLAGS_kudu_operation_timeout_ms);
        JniLocalFrame jni_frame; <---
        RETURN_IF_ERROR(jni_frame.push(jni_env));  <----
        RETURN_IF_ERROR(SerializeThriftMsg(jni_env, &cfg, cfg_bytes));
        return Status::OK();
      

      We use a local frame that could be GC'ed by the JVM. There is no need to use it. This was found in testing where we instantiate multiple fe instances.

        Activity

        Show
        bharathv bharath v added a comment - Fixed via https://github.com/apache/incubator-impala/commit/ba2613abfb39357843e0a6e5dfd2d5ed49b36ea3

          People

          • Assignee:
            bharathv bharath v
            Reporter:
            bharathv bharath v
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development