Repository: tajo
Updated Branches:
  refs/heads/branch-0.11.1 1ccd0eb49 -> 8ddefef85


TAJO-1858: Aligning error message in execute query page of web UI is needed.

Signed-off-by: Jihoon Son <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/tajo/repo
Commit: http://git-wip-us.apache.org/repos/asf/tajo/commit/8ddefef8
Tree: http://git-wip-us.apache.org/repos/asf/tajo/tree/8ddefef8
Diff: http://git-wip-us.apache.org/repos/asf/tajo/diff/8ddefef8

Branch: refs/heads/branch-0.11.1
Commit: 8ddefef858746501c4c2bf3dfc61484a2f38b658
Parents: 1ccd0eb
Author: Byunghwa Yun <[email protected]>
Authored: Thu Dec 24 15:28:31 2015 +0900
Committer: Jihoon Son <[email protected]>
Committed: Thu Dec 24 15:28:31 2015 +0900

----------------------------------------------------------------------
 CHANGES                                         |  3 +++
 .../resources/webapps/admin/query_executor.jsp  | 20 ++++++++++----------
 2 files changed, 13 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tajo/blob/8ddefef8/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 933bc4d..a3c55f7 100644
--- a/CHANGES
+++ b/CHANGES
@@ -7,6 +7,9 @@ Release 0.11.1 - unreleased
 
   IMPROVEMENT
 
+    TAJO-1858: Aligning error message in execute query page of web UI is 
needed.
+    (Byunghwa Yun via jihoon)
+
     TAJO-1979: Usage of some TajoShellCommand is omitted.
     (Contributed by Dongkyu Hwangbo, committed by jaehwa)
 

http://git-wip-us.apache.org/repos/asf/tajo/blob/8ddefef8/tajo-core/src/main/resources/webapps/admin/query_executor.jsp
----------------------------------------------------------------------
diff --git a/tajo-core/src/main/resources/webapps/admin/query_executor.jsp 
b/tajo-core/src/main/resources/webapps/admin/query_executor.jsp
index cc3efce..6b6a3db 100644
--- a/tajo-core/src/main/resources/webapps/admin/query_executor.jsp
+++ b/tajo-core/src/main/resources/webapps/admin/query_executor.jsp
@@ -147,7 +147,7 @@ function clearTimer() {
 
 function getQueryStatusHtml(status) {
   if(status.success == "false") {
-    return "<div style='color:#ff0000; margin-top: 5px'>" + 
status.errorMessage + "</div>";
+    return "<div style='color:#ff0000; margin-top: 5px'><pre>" + 
status.errorMessage + "</pre></div>";
   } else {
     var statusHtml = "<div style='margin-top: 5px'>Start: " + status.startTime 
+ "</div>";
     statusHtml += "<div style='margin-top: 5px'>Finish: " + status.finishTime 
+ "</div>";
@@ -173,11 +173,11 @@ function getResult() {
     $("#queryResult").html("");
     var resultColumns = resultJson.resultColumns;
     var resultData = resultJson.resultData;
-       
+
     storedColumns = resultColumns;
-    storedData = resultData; 
+    storedData = resultData;
     pageCount = Math.ceil((storedData.length / PRINT_LIMIT)) - 1 ;
-       
+
     var resultTable = "<table width='100%' class='border_table'><tr>";
     for(var i = 0; i < resultColumns.length; i++) {
       resultTable += "<th>" + resultColumns[i] + "</th>";
@@ -188,7 +188,7 @@ function getResult() {
       for(var j = 0; j < resultData[i].length; j++) {
         resultTable += "<td>" + resultData[i][j] + "</td>";
       }
-      resultTable += "</tr>";    
+      resultTable += "</tr>";
       if(++printedLine >= PRINT_LIMIT) break;
     }
     resultTable += "</table>";
@@ -213,7 +213,7 @@ function getCSV() {
   var colCount = storedColumns.length;
   for(var colIndex = 0; colIndex < colCount; colIndex++) {
     if(colIndex == 0) {
-      csvData += storedColumns[colIndex];  
+      csvData += storedColumns[colIndex];
     } else {
       csvData += "," + storedColumns[colIndex];
     }
@@ -226,15 +226,15 @@ function getCSV() {
       } else {
         csvData += "," + storedData[rowIndex][colIndex];
       }
-    }   
+    }
     csvData += "\n";
   }
   $("#csvData").val(csvData);
-  $("#dataForm").submit();  
+  $("#dataForm").submit();
 }
 
 function getNext() {
-       var printedLine = 0;    
+       var printedLine = 0;
        if(pageCount > pageNum) {
                pageNum++;
                document.getElementById("selectPage").options.selectedIndex = 
pageNum;
@@ -295,7 +295,7 @@ function getPage() {
   <h2>Tajo Master: <%=masterLabel%> <%=activeLabel%></h2>
   <hr/>
   <h3>Query</h3>
-  Database :  
+  Database :
   <select id="selectDatabase" name="database" width="190" style="width: 190px">
     <%
        for (String databaseName : master.getCatalog().getAllDatabaseNames()) {

Reply via email to