Better handle Presto errors.

NREL · Mar 23, 2016 · 455eaf0 · 455eaf0
1 parent 6284a9f
commit 455eaf0
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/api-umbrella/web-app/app/models/log_search/kylin.rb b/src/api-umbrella/web-app/app/models/log_search/kylin.rb
@@ -47,6 +47,11 @@ def execute_presto(sql)
     }
     while(response.status == 200)
       query_result = MultiJson.load(response.body)
+      if(query_result["stats"] && query_result["stats"]["state"] == "FAILED")
+        Rails.logger.error(response.body)
+        raise "Presto Error"
+      end
+
       if(results["columnMetas"].empty? && query_result["columns"])
         results["columnMetas"] = query_result["columns"].map do |column|
           {