-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20653/
-----------------------------------------------------------

(Updated April 26, 2014, 4:10 p.m.)


Review request for Tajo.


Changes
-------

I fixed a bug that Hyunsik pointed.


Bugs: TAJO-766
    https://issues.apache.org/jira/browse/TAJO-766


Repository: tajo


Description
-------

In this test, the console message is checked, but it can be different according 
to the system's language.
For example, the test is failed in my desktop because the console message is in 
Korean.

To handle this problem, I modified TajoCli.executeMetaCommand() to return -1 
when an exception occurs.
In addition, I fixed a bug in TajoCli, and improved ExecExternalSheelCommand to 
print invalid commands.


Diffs (updated)
-----

  tajo-client/src/main/java/org/apache/tajo/cli/ExecExternalShellCommand.java 
ea2db28 
  tajo-client/src/main/java/org/apache/tajo/cli/TajoCli.java 606ca88 
  tajo-core/src/test/java/org/apache/tajo/cli/TestExecExternalShellCommand.java 
9c6e760 
  tajo-rpc/src/test/java/org/apache/tajo/rpc/TestBlockingRpc.java 7acede6 

Diff: https://reviews.apache.org/r/20653/diff/


Testing
-------

mvn clean install


Thanks,

Jihoon Son

Reply via email to