You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@impala.apache.org by "bharath v (JIRA)" <ji...@apache.org> on 2017/05/14 17:38:04 UTC

[jira] [Resolved] (IMPALA-5322) Potential crash in Frontend & Catalog JNI startup

     [ https://issues.apache.org/jira/browse/IMPALA-5322?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

bharath v resolved IMPALA-5322.
-------------------------------
    Resolution: Fixed

Fixed via https://github.com/apache/incubator-impala/commit/ba2613abfb39357843e0a6e5dfd2d5ed49b36ea3

> Potential crash in Frontend & Catalog JNI startup
> -------------------------------------------------
>
>                 Key: IMPALA-5322
>                 URL: https://issues.apache.org/jira/browse/IMPALA-5322
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Backend
>    Affects Versions: Impala 2.8.0
>            Reporter: bharath v
>            Assignee: bharath v
>            Priority: Critical
>             Fix For: Impala 2.9.0
>
>
> {noformat}
>  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();
> {noformat}
> 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.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)