AMBARI-17727. removed comment of --Global Settings-- from query and corrected addition of semi colon at end (nitirajrathore)

contrib/views/hive-next/src/main/resources/ui/hive-web/app/controllers/index.js

@@ -199,7 +199,6 @@ export default Ember.Controller.extend({
     finalQuery = query;
     finalQuery = this.bindQueryParams(finalQuery);
     finalQuery = this.prependGlobalSettings(finalQuery, job);
-    finalQuery = this.rearrangeComponents(finalQuery);
     job.set('forcedContent', finalQuery);
 
     if (shouldGetVisualExplain) {
@@ -209,25 +208,6 @@ export default Ember.Controller.extend({
     return this.createJob(job, originalModel);
   },
 
-  /**
-   * Rearranges the files and udfs statements to the starting of the query.
-   */
-  rearrangeComponents: function(query) {
-    var extractedComponents = this.extractComponents(query);
-    var modifiedFinalQuery = '';
-    if (extractedComponents.files.length) {
-      modifiedFinalQuery += extractedComponents.files.join("\n") + "\n\n";
-    }
-
-    if (extractedComponents.udfs.length) {
-      modifiedFinalQuery += extractedComponents.udfs.join("\n") + "\n\n";
-    }
-    var newQueries = extractedComponents.queryString.split(";");
-    newQueries = newQueries.filter(Boolean);
-    modifiedFinalQuery += newQueries.join(";") + ";";
-    return modifiedFinalQuery;
-  },
-
   getVisualExplainJson: function (job, originalModel) {
     var self = this;
     var defer = Ember.RSVP.defer();
@@ -347,7 +327,11 @@ export default Ember.Controller.extend({
       finalQuery += queryComponents.udfs.join("\n") + "\n\n";
     }
 
-    finalQuery += queries.join(";") + ";";
+    finalQuery += queries.join(";");
+    if(!finalQuery.trim().endsWith(';')){
+      finalQuery = finalQuery.trim() + ";";
+    }
+
     return finalQuery.trim();
   },
 

contrib/views/hive-next/src/main/resources/ui/hive-web/app/services/settings.js

@@ -172,8 +172,6 @@ export default Ember.Service.extend({
       asString += "set %@=%@;\n".fmt(setting.get('key.name'), setting.get('value'));
     });
 
-    asString += constants.globalSettings.comment;
-
     return asString;
   },
 

contrib/views/hive-next/src/main/resources/ui/hive-web/app/utils/constants.js

@@ -224,10 +224,6 @@ export default Ember.Object.create({
     }
   },
 
-  globalSettings: {
-    comment: "--Global Settings--\n\n"
-  },
-
   defaultVisualizationRowCount: 10000
 
 });

contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index.js

@@ -200,7 +200,6 @@ export default Ember.Controller.extend({
     finalQuery = query;
     finalQuery = this.bindQueryParams(finalQuery);
     finalQuery = this.prependGlobalSettings(finalQuery, job);
-    finalQuery = this.rearrangeComponents(finalQuery);
     job.set('forcedContent', finalQuery);
 
     if (shouldGetVisualExplain) {
@@ -210,25 +209,6 @@ export default Ember.Controller.extend({
     return this.createJob(job, originalModel);
   },
 
-  /**
-   * Rearranges the files and udfs statements to the starting of the query.
-   */
-  rearrangeComponents: function(query) {
-    var extractedComponents = this.extractComponents(query);
-    var modifiedFinalQuery = '';
-    if (extractedComponents.files.length) {
-      modifiedFinalQuery += extractedComponents.files.join("\n") + "\n\n";
-    }
-
-    if (extractedComponents.udfs.length) {
-      modifiedFinalQuery += extractedComponents.udfs.join("\n") + "\n\n";
-    }
-    var newQueries = extractedComponents.queryString.split(";");
-    newQueries = newQueries.filter(Boolean);
-    modifiedFinalQuery += newQueries.join(";") + ";";
-    return modifiedFinalQuery;
-  },
-
   getVisualExplainJson: function (job, originalModel) {
     var self = this;
     var defer = Ember.RSVP.defer();
@@ -338,7 +318,11 @@ export default Ember.Controller.extend({
       finalQuery += queryComponents.udfs.join("\n") + "\n\n";
     }
 
-    finalQuery += queries.join(";") + ";";
+    finalQuery += queries.join(";");
+    if(!finalQuery.trim().endsWith(';')){
+      finalQuery = finalQuery.trim() + ";";
+    }
+
     return finalQuery.trim();
   },
 

contrib/views/hive/src/main/resources/ui/hive-web/app/services/settings.js

@@ -172,8 +172,6 @@ export default Ember.Service.extend({
       asString += "set %@=%@;\n".fmt(setting.get('key.name'), setting.get('value'));
     });
 
-    asString += constants.globalSettings.comment;
-
     return asString;
   },
 

contrib/views/hive/src/main/resources/ui/hive-web/app/utils/constants.js

@@ -225,10 +225,6 @@ export default Ember.Object.create({
     }
   },
 
-  globalSettings: {
-    comment: "--Global Settings--\n\n"
-  },
-
   defaultVisualizationRowCount: 10000
 
 });