[
https://issues.apache.org/jira/browse/TAJO-1228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14257933#comment-14257933
]
ASF GitHub Bot commented on TAJO-1228:
--------------------------------------
GitHub user hyunsik opened a pull request:
https://github.com/apache/tajo/pull/317
TAJO-1228: TajoClient should communicate with only TajoMaster without
TajoWorker.
See the description at https://issues.apache.org/jira/browse/TAJO-1228.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/hyunsik/tajo TAJO-1228
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tajo/pull/317.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #317
----
commit 64fb64ea9b3de3681a22c9f64e90beff06d001e1
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-21T11:23:17Z
Initial work
commit 0241bbcd86517604cc997471860b1f5af4a2f4b5
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-21T12:19:24Z
All test passed after getStatus from TajoMaster.
commit 01a1aed280b0007eb2d07b6548660e7eba18d12a
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-21T15:37:37Z
Completely remove connection of query master of query status.
commit a8983b73911f8c25cac270f04137101d2cb2930d
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-24T02:16:25Z
Passed all unit tests and removed worker client service.
commit 546837ee8176ebecc38f14050fdbc0394ea0fd43
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-24T03:15:37Z
Remove queryMasterMap from SessionConnection.
commit e158fe72fecb111fbb7b72f8c76397901b6762a2
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-24T03:47:34Z
Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into
TAJO-1228
Conflicts:
tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
tajo-core/src/test/java/org/apache/tajo/TajoTestingCluster.java
commit 9de88de5da0a27145ae3626ed2c9db47b0d8092d
Author: Hyunsik Choi <[email protected]>
Date: 2014-12-24T04:03:59Z
Clean up codes.
----
> TajoClient should communicate with only TajoMaster without TajoWorker
> ---------------------------------------------------------------------
>
> Key: TAJO-1228
> URL: https://issues.apache.org/jira/browse/TAJO-1228
> Project: Tajo
> Issue Type: Bug
> Components: client
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.10
>
>
> Currently, TajoClient connects with TajoMaster as well as TajoWorker. A query
> submission is passed from TajoClient to TajoMaster, and than TajoMaster
> forwards the query to one query master running on a TajoWorker. After than, a
> client monitors the query progress through the query master.
> In other words, TajoClient contacts both TajoMaster and TajoWorker. It has
> three disadvantages:
> * a network firewall should allow connections to TajoWorker nodes.
> * all components should have complex states and communication logic one
> another.
> * TAJO-1160 is trying to remove hadoop dependency from TajoClient. In
> TAJO-1160, TajoClient will use only some server which forwards query results
> to itself instead of reading directly HDFS. TajoMaster would be the best
> component to forward the query results because TajoClient always connects
> with TajoMaster. So, if TajoClient communicates with only TajoMaster, all
> client logic and protocol would be simple.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
