[TAJO-960] TajoCli's problem does not show the current status

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 0.9.0
• Assignee: Hyunsik Choi
• Reporter: Hyunsik Choi
• Created: 18/Jul/14 07:59
• Updated: 20/Jul/14 13:00
• Resolved: 20/Jul/14 12:48

Description

TajoCli is designed to show the status of completeness of SQL statements.

For example, it should work as follows:

tpch> CREATE TABLE T1 (
> l_orderkey INT,
> l_partkey INT)
> 
Created T1

tpch> SELECT 
> t1,
> ..

But, the current TajoCli does not handle this status. So, it works as follows:

tpch> CREATE TABLE T1 (
tpch> t1 int,
tpch> t2 int);
Created T1
tpch>

Users cannot know if current status continues to take statements or is a ready status to start new statement.

The solution is to place executeParsedResults(parsedResults); and currentPrompt = updatePrompt(parser.getState()); to outside of if (parsedResults.size() > 0) statement.

if (parsedResults.size() > 0) {
          for (ParsedResult parsed : parsedResults) {
            history.addStatement(parsed.getHistoryStatement() + (parsed.getType() == STATEMENT ? ";" : ""));
          }
          executeParsedResults(parsedResults);
          currentPrompt = updatePrompt(parser.getState());
        }