> On Sept. 17, 2014, 3:27 p.m., Hyunsik Choi wrote:
> > If you are not working on my suggestion and you are Ok, I'd like to add my 
> > change to this patch. It would be helpful to complete this issue.

I am working on your suggestion. Currently, the above FileNotFound exception 
has been solved and the query result looks good. I will upload a new patch 
soon. However, if you have already had a new patch with correct query results, 
do not hesitate to upload the patch and I will review. I guess that you want to 
resolve this issue before the release of version 0.9.0, right ?


- Hai Thanh


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


On Sept. 4, 2014, 8:40 a.m., Hai Thanh Mai wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25242/
> -----------------------------------------------------------
> 
> (Updated Sept. 4, 2014, 8:40 a.m.)
> 
> 
> Review request for Tajo and Hyunsik Choi.
> 
> 
> Repository: tajo
> 
> 
> Description
> -------
> 
> TAJO-983: Worker should directly read Intermediate data stored in localhost 
> rather than fetching
> 
> Currently, worker always fetches all intermediate via Fetcher and than store 
> them in local file system even though some intermediate data already are 
> stored in local file system. It is inefficient and causes unnecessary I/O and 
> extra storage occupation. We should improve it.
> 
> https://issues.apache.org/jira/browse/TAJO-983
> 
> 
> Diffs
> -----
> 
>   tajo-core/src/main/java/org/apache/tajo/worker/Fetcher.java 64475fe 
>   tajo-core/src/main/java/org/apache/tajo/worker/Task.java d0665ae 
>   tajo-core/src/test/java/org/apache/tajo/worker/TestFetcher.java 95c06bb 
>   
> tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
>  150ac85 
>   
> tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java
>  a8b424e 
> 
> Diff: https://reviews.apache.org/r/25242/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hai Thanh Mai
> 
>

Reply via email to