This is an automated email from the ASF dual-hosted git repository.

maximebeauchemin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 774ad45  [bugfix] capture Hive job_id pre-url transformation (#3213)
774ad45 is described below

commit 774ad45efbaad86238004e9a21d4c733f57bde36
Author: Maxime Beauchemin <maximebeauche...@gmail.com>
AuthorDate: Mon Jul 31 22:22:08 2017 -0700

    [bugfix] capture Hive job_id pre-url transformation (#3213)
---
 superset/db_engine_specs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/db_engine_specs.py b/superset/db_engine_specs.py
index b43e94e..f159c7c 100644
--- a/superset/db_engine_specs.py
+++ b/superset/db_engine_specs.py
@@ -751,13 +751,13 @@ class HiveEngineSpec(PrestoEngineSpec):
                 if not tracking_url:
                     tracking_url = cls.get_tracking_url(log_lines)
                     if tracking_url:
+                        job_id = tracking_url.split('/')[-2]
                         logging.info(
                             "Found the tracking url: {}".format(tracking_url))
                         tracking_url = tracking_url_trans(tracking_url)
                         logging.info(
                             "Transformation applied: {}".format(tracking_url))
                         query.tracking_url = tracking_url
-                        job_id = tracking_url.split('/')[-2]
                         logging.info("Job id: {}".format(job_id))
                         needs_commit = True
                 if job_id and len(log_lines) > last_log_line:

-- 
To stop receiving notification emails like this one, please contact
['"comm...@superset.apache.org" <comm...@superset.apache.org>'].

Reply via email to