You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by hyoungjun kim <ba...@gmail.com> on 2014/04/10 17:26:45 UTC

Review Request 20217: Arrange TajoCli output message.

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

Review request for Tajo.


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


Repository: tajo


Description
-------

See title. TajoCli prints information and error message with various format. These message must be arranged because ETL program uses TajoCli's information or error message.


Diffs
-----

  tajo-client/src/main/java/org/apache/tajo/cli/DefaultTajoCliOutputFormatter.java PRE-CREATION 
  tajo-client/src/main/java/org/apache/tajo/cli/TajoCli.java c277c4c 
  tajo-client/src/main/java/org/apache/tajo/cli/TajoCliOutputFormatter.java PRE-CREATION 
  tajo-client/src/main/java/org/apache/tajo/client/QueryStatus.java 203f9aa 
  tajo-client/src/main/java/org/apache/tajo/client/TajoClient.java 81fc227 
  tajo-client/src/main/proto/ClientProtos.proto 3baeee9 
  tajo-common/src/main/java/org/apache/tajo/conf/TajoConf.java 3c81ed5 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/GlobalEngine.java cbb8f37 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/TajoMasterClientService.java c968a73 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/Query.java a8f5b31 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryInfo.java b077b36 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryMasterManagerService.java bf59e9f 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryMasterTask.java 8b3403f 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryUnit.java d0fde4f 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryUnitAttempt.java 7993ce9 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java 63b50ac 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/worker/TajoWorkerClientService.java 937d886 
  tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/worker/Task.java 30f56ee 
  tajo-core/tajo-core-backend/src/main/proto/TajoWorkerProtocol.proto 7c94e33 
  tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/cli/TestDefaultCliOutputFormatter.java PRE-CREATION 
  tajo-dist/src/main/bin/tajo 24dce60 
  tajo-dist/src/main/bin/tajo-daemon.sh 1d4a22b 
  tajo-dist/src/main/conf/log4j.properties f46fffb 
  tajo-dist/src/main/conf/tajo-env.sh e61415c 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcClient.java 604ed52 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcServer.java 067d824 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/NettyClientBase.java 8f49e17 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/RemoteCallException.java 6504e69 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/RpcChannelFactory.java adafd5c 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/TajoServiceException.java PRE-CREATION 
  tajo-rpc/src/main/proto/RpcProtos.proto 2dac58d 
  tajo-rpc/src/test/java/org/apache/tajo/rpc/TestBlockingRpc.java fe8685a 

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


Testing
-------

mvn clean install


Thanks,

hyoungjun kim

Re: Review Request 20217: TAJO-734: Arrange TajoCli output message.

Posted by Jihoon Son <ji...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20217/#review41115
-----------------------------------------------------------

Ship it!


+1.
Thanks for your contribution!

- Jihoon Son


On April 23, 2014, 12:03 p.m., hyoungjun kim wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20217/
> -----------------------------------------------------------
> 
> (Updated April 23, 2014, 12:03 p.m.)
> 
> 
> Review request for Tajo.
> 
> 
> Bugs: TAJO-734
>     https://issues.apache.org/jira/browse/TAJO-734
> 
> 
> Repository: tajo
> 
> 
> Description
> -------
> 
> See title. TajoCli prints information and error message with various format. These message must be arranged because ETL program uses TajoCli's information or error message.
> 
> 
> Diffs
> -----
> 
>   tajo-client/src/main/java/org/apache/tajo/cli/DefaultTajoCliOutputFormatter.java PRE-CREATION 
>   tajo-client/src/main/java/org/apache/tajo/cli/TajoCli.java 508b8bb 
>   tajo-client/src/main/java/org/apache/tajo/cli/TajoCliOutputFormatter.java PRE-CREATION 
>   tajo-client/src/main/java/org/apache/tajo/client/QueryStatus.java 203f9aa 
>   tajo-client/src/main/java/org/apache/tajo/client/TajoClient.java 81fc227 
>   tajo-client/src/main/proto/ClientProtos.proto 3baeee9 
>   tajo-common/src/main/java/org/apache/tajo/conf/TajoConf.java 5b3d4b3 
>   tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java 8acf2b2 
>   tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java c968a73 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/Query.java a8f5b31 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryInfo.java b077b36 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryMasterManagerService.java bf59e9f 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryMasterTask.java 39ea430 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryUnit.java 42fbf8a 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryUnitAttempt.java 7993ce9 
>   tajo-core/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java 8929e8d 
>   tajo-core/src/main/java/org/apache/tajo/worker/TajoWorkerClientService.java 937d886 
>   tajo-core/src/main/java/org/apache/tajo/worker/Task.java 30f56ee 
>   tajo-core/src/main/proto/TajoWorkerProtocol.proto 7c94e33 
>   tajo-core/src/test/java/org/apache/tajo/cli/TestDefaultCliOutputFormatter.java PRE-CREATION 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcClient.java 604ed52 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcServer.java 067d824 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/NettyClientBase.java 8f49e17 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/RemoteCallException.java 6504e69 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/RpcChannelFactory.java adafd5c 
>   tajo-rpc/src/main/java/org/apache/tajo/rpc/TajoServiceException.java PRE-CREATION 
>   tajo-rpc/src/main/proto/RpcProtos.proto 2dac58d 
>   tajo-rpc/src/test/java/org/apache/tajo/rpc/TestBlockingRpc.java fe8685a 
> 
> Diff: https://reviews.apache.org/r/20217/diff/
> 
> 
> Testing
> -------
> 
> mvn clean install
> 
> 
> Thanks,
> 
> hyoungjun kim
> 
>

Re: Review Request 20217: TAJO-734: Arrange TajoCli output message.

Posted by hyoungjun kim <ba...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20217/
-----------------------------------------------------------

(Updated April 23, 2014, 3:03 a.m.)


Review request for Tajo.


Changes
-------

Rebased


Summary (updated)
-----------------

TAJO-734: Arrange TajoCli output message.


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


Repository: tajo


Description
-------

See title. TajoCli prints information and error message with various format. These message must be arranged because ETL program uses TajoCli's information or error message.


Diffs (updated)
-----

  tajo-client/src/main/java/org/apache/tajo/cli/DefaultTajoCliOutputFormatter.java PRE-CREATION 
  tajo-client/src/main/java/org/apache/tajo/cli/TajoCli.java 508b8bb 
  tajo-client/src/main/java/org/apache/tajo/cli/TajoCliOutputFormatter.java PRE-CREATION 
  tajo-client/src/main/java/org/apache/tajo/client/QueryStatus.java 203f9aa 
  tajo-client/src/main/java/org/apache/tajo/client/TajoClient.java 81fc227 
  tajo-client/src/main/proto/ClientProtos.proto 3baeee9 
  tajo-common/src/main/java/org/apache/tajo/conf/TajoConf.java 5b3d4b3 
  tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java 8acf2b2 
  tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java c968a73 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/Query.java a8f5b31 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryInfo.java b077b36 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryMasterManagerService.java bf59e9f 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryMasterTask.java 39ea430 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryUnit.java 42fbf8a 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/QueryUnitAttempt.java 7993ce9 
  tajo-core/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java 8929e8d 
  tajo-core/src/main/java/org/apache/tajo/worker/TajoWorkerClientService.java 937d886 
  tajo-core/src/main/java/org/apache/tajo/worker/Task.java 30f56ee 
  tajo-core/src/main/proto/TajoWorkerProtocol.proto 7c94e33 
  tajo-core/src/test/java/org/apache/tajo/cli/TestDefaultCliOutputFormatter.java PRE-CREATION 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcClient.java 604ed52 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/BlockingRpcServer.java 067d824 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/NettyClientBase.java 8f49e17 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/RemoteCallException.java 6504e69 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/RpcChannelFactory.java adafd5c 
  tajo-rpc/src/main/java/org/apache/tajo/rpc/TajoServiceException.java PRE-CREATION 
  tajo-rpc/src/main/proto/RpcProtos.proto 2dac58d 
  tajo-rpc/src/test/java/org/apache/tajo/rpc/TestBlockingRpc.java fe8685a 

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


Testing
-------

mvn clean install


Thanks,

hyoungjun kim