Hi Chris, Please wait for the test. You should apply TAJO-1048 too before testing. TAJO-1048 seems be committed shortly.
Best regards, Hyunsik On Thu, Sep 18, 2014 at 12:41 AM, Hyunsik Choi <[email protected]> wrote: > Hi Chris, > > I'm expecting that your problem is caused by the legacy limitation of > parallel degree. If you apply this patch > (https://issues.apache.org/jira/browse/TAJO-1049) and set the session > variables as I mentioned, the number of join tasks will be increased. > Could you please test it and share the result? > > Warm regards, > Hyunsik > > On Wed, Sep 17, 2014 at 11:55 PM, Christian Schwabe > <[email protected]> wrote: >> >> Hello Hyunsik, >> >> thanks for your fast reply and the explanation how to apply a patch. >> Unfortunately the patch does not resolve my problem and my query does not >> finished. >> Any other further ideas? >> >> Best regards, >> Chris >> >> >> >> Am 17.09.2014 16:39:14, schrieb Hyunsik Choi: >> >> Hi Chris, >> >> Do you mean that your query is finished well? Good to hear. But, you >> need to know that the patch still has some bugs. Please see the review >> https://reviews.apache.org/r/25242/. >> >> If your problem is solved by this patch, your problem was related to >> Fetcher. I'm also expecting your problem is related to this issue >> (https://github.com/apache/tajo/pull/141/files), which already >> committed to master branch. You can use this fix from master branch >> too. >> >> Please refer to this document in order to learn how to apply patch. We >> mostly use '--no-prefix' option in order to generate patch files. So, >> you should use the patch command with -p0 instead of -p1 as follows: >> >> patch -p0 < ${PATCH_FILE}.patch >> >> By default, patch command uses -p1. >> >> Thanks, >> Hyunsik >> >> On Wed, Sep 17, 2014 at 8:07 AM, Christian Schwabe >> <[email protected]> wrote: >> >> Hello guys, >> >> after days of efforts to apply the patch from May Hai, I finally be able to >> announce a success today. I want to a brief description where the problem >> was. The few guides out there to apply a patch, using the Git clients often >> use git apply. However, I had no success. See discussion below. >> >> christians-mbp:tajo chris$ git apply --check >> /Users/chris/Desktop/TAJO-983.140916.patch >> error: src/main/java/org/apache/tajo/worker/Fetcher.java: No such file or >> directory >> error: src/main/java/org/apache/tajo/worker/Task.java: No such file or >> directory >> error: >> src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java: No >> such file or directory >> error: src/test/java/org/apache/tajo/worker/TestFetcher.java: No such file >> or directory >> error: src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java: >> No such file or directory >> error: src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java: No >> such file or directory >> >> ########################################### >> >> christians-mbp:tajo chris$ git apply --check >> /Users/chris/Desktop/TAJO-983.140916.patch >> error: src/main/java/org/apache/tajo/worker/Fetcher.java: No such file or >> directory >> error: src/main/java/org/apache/tajo/worker/Task.java: No such file or >> directory >> error: >> src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java: No >> such file or directory >> error: src/test/java/org/apache/tajo/worker/TestFetcher.java: No such file >> or directory >> error: src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java: >> No such file or directory >> error: src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java: No >> such file or directory >> >> ########################################### >> >> christians-mbp:tajo chris$ git apply -v --check >> /Users/chris/Desktop/TAJO-983.140916.patch >> Checking patch src/main/java/org/apache/tajo/worker/Fetcher.java... >> error: src/main/java/org/apache/tajo/worker/Fetcher.java: No such file or >> directory >> Checking patch src/main/java/org/apache/tajo/worker/Task.java... >> error: src/main/java/org/apache/tajo/worker/Task.java: No such file or >> directory >> Checking patch >> src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java... >> error: >> src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java: No >> such file or directory >> Checking patch src/test/java/org/apache/tajo/worker/TestFetcher.java... >> error: src/test/java/org/apache/tajo/worker/TestFetcher.java: No such file >> or directory >> Checking patch >> src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java... >> error: src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java: >> No such file or directory >> Checking patch >> src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java... >> error: src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java: No >> such file or directory >> >> ########################################### >> >> christians-mbp:tajo-core chris$ git apply >> /Users/chris/Desktop/TAJO-983.140916.patch >> /Users/chris/Desktop/TAJO-983.140916.patch:58: trailing whitespace. >> >> >> >> /Users/chris/Desktop/TAJO-983.140916.patch:80: trailing whitespace. >> >> >> >> /Users/chris/Desktop/TAJO-983.140916.patch:487: trailing whitespace. >> return " (start=" + startOffset() + ", length=" + length + ", >> fromRemote=" + fromRemote >> warning: 3 lines add whitespace errors. >> >> ########################################### >> >> Removal of whitespaces also brought no success. >> My hope depended least on this patch. As far Tajo not run in my environment >> and my standard configuration, I will unfortunately have to complete my >> thesis with the result that it is currently not suitable for practical use. >> This really is not a reproach to formulate that clear again. I really do >> appreciate your hard work very much. However, I also think it's slightly >> doubtful that the advice you have given can't read anywhere and only >> inquired. I think that's still a pity. Surely you can not really expect, or >> rather the can but not really your goal to be always be the first point of >> contact. I think at this point there is still a lot of catching up to do. >> This is still a large deficit. Surely you can not really expect, or rather >> it is hopefully not really your goal to be always be the first point of >> contact. I think at this point there is still a lot of catching up to do. >> >> I am not come within the scope of my thesis within the time to the desired >> goal, but of course I'm still interested in a close contact with you and and >> a further effort to solve the outstanding problem. >> >> Until I have encountered the following with the patch was successfully >> carried out. Unfortunately, this patch did not bring the hoped solution. No >> change in the behavior described herein. Are there any other ideas that I >> could try to fix the problem? >> >> I really thank you very much for your support thus far. I really thank you >> very much for your support thus far. I really thank you very much for your >> support thus far. I really thank you very much for your support thus far. I >> really thank you very much for your support thus far. I really thank you >> very much for your support thus far. I really thank you very much for your >> support thus far. I really thank you very much for your support thus far. >> ;-) >> >> >> christians-mbp:tajo chris$ cd tajo-yarn-pullserver/ >> christians-mbp:tajo-yarn-pullserver chris$ patch -p1 < >> ../TAJO-983.140916.patch >> can't find file to patch at input line 5 >> Perhaps you used the wrong -p or --strip option? >> The text leading up to this was: >> -------------------------- >> |diff --git tajo-core/src/main/java/org/apache/tajo/worker/Fetcher.java >> tajo-core/src/main/java/org/apache/tajo/worker/Fetcher.java >> |index 64475fe..7310736 100644 >> |--- tajo-core/src/main/java/org/apache/tajo/worker/Fetcher.java >> |+++ tajo-core/src/main/java/org/apache/tajo/worker/Fetcher.java >> -------------------------- >> File to patch: >> Skip this patch? [y] y >> Skipping patch. >> 6 out of 6 hunks ignored >> can't find file to patch at input line 98 >> Perhaps you used the wrong -p or --strip option? >> The text leading up to this was: >> -------------------------- >> |diff --git tajo-core/src/main/java/org/apache/tajo/worker/Task.java >> tajo-core/src/main/java/org/apache/tajo/worker/Task.java >> |index 7b4cbe1..ecc4078 100644 >> |--- tajo-core/src/main/java/org/apache/tajo/worker/Task.java >> |+++ tajo-core/src/main/java/org/apache/tajo/worker/Task.java >> -------------------------- >> File to patch: y >> y: No such file or directory >> Skip this patch? [y] y >> Skipping patch. >> 7 out of 7 hunks ignored >> can't find file to patch at input line 339 >> Perhaps you used the wrong -p or --strip option? >> The text leading up to this was: >> -------------------------- >> |diff --git >> tajo-core/src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java >> tajo-core/src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java >> |index 4f11168..911c0c2 100644 >> |--- >> tajo-core/src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java >> |+++ >> tajo-core/src/main/java/org/apache/tajo/worker/dataserver/retriever/FileChunk.java >> -------------------------- >> File to patch: y >> y: No such file or directory >> Skip this patch? [y] y >> Skipping patch. >> 1 out of 1 hunk ignored >> can't find file to patch at input line 362 >> Perhaps you used the wrong -p or --strip option? >> The text leading up to this was: >> -------------------------- >> |diff --git tajo-core/src/test/java/org/apache/tajo/worker/TestFetcher.java >> tajo-core/src/test/java/org/apache/tajo/worker/TestFetcher.java >> |index b755e02..cb19d54 100644 >> |--- tajo-core/src/test/java/org/apache/tajo/worker/TestFetcher.java >> |+++ tajo-core/src/test/java/org/apache/tajo/worker/TestFetcher.java >> -------------------------- >> File to patch: y >> y: No such file or directory >> Skip this patch? [y] y >> Skipping patch. >> 6 out of 6 hunks ignored >> patching file >> src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java >> patching file >> src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java >> christians-mbp:tajo-yarn-pullserver chris$ pwd >> /Users/chris/6_Semester/Thesis/07_git/tajo/tajo-yarn-pullserver >> >> >> Warm regards, >> Chris >> >>
