Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-1846

Python temp directory path should be selected differently based on user platform

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.11.0
    • Component/s: None
    • Labels:
      None

      Description

      in PythonScriptEngine, Some of variable has no consideration in user's os.
      When I run tajo with Windows 7, tajo gives me this log message.

      2015-09-14 10:20:35,592 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
      2015-09-14 10:20:37,336 INFO org.apache.tajo.master.TajoMaster: FileSystem (hdfs://localhost:9000) is initialized.
      2015-09-14 10:20:37,740 INFO org.apache.tajo.master.TajoMaster: Tajo Warehouse dir: hdfs://localhost:9000/tajo/tajo/warehouse
      2015-09-14 10:20:37,742 INFO org.apache.tajo.master.TajoMaster: Staging dir: hdfs://localhost:9000/tajo/tajo/warehouse
      2015-09-14 10:20:41,581 INFO org.apache.tajo.plan.function.python.PythonScriptEngine: Python base dir is C:\Users\dk\AppData\Local\Temp\/tajo-dk/python
      2015-09-14 10:20:41,584 INFO org.apache.hadoop.service.AbstractService: Service org.apache.tajo.master.TajoMaster failed in state INITED; cause: java.lang.NullPointerException
      java.lang.NullPointerException
      

        Activity

        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-0.11.0-build #50 (See https://builds.apache.org/job/Tajo-0.11.0-build/50/)
        TAJO-1846: Python temp directory path should be selected differently based on user platform. (jihoonson: rev 4ac2a4c4a4e68be4c8d5efd89f571c088411b354)

        • tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java
        • CHANGES
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-0.11.0-build #50 (See https://builds.apache.org/job/Tajo-0.11.0-build/50/ ) TAJO-1846 : Python temp directory path should be selected differently based on user platform. (jihoonson: rev 4ac2a4c4a4e68be4c8d5efd89f571c088411b354) tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java CHANGES
        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-build #873 (See https://builds.apache.org/job/Tajo-master-build/873/)
        TAJO-1846: Python temp directory path should be selected differently based on user platform. (jihoonson: rev 8ef49f0186ce813d44ba997b0fdde91497c87cda)

        • CHANGES
        • tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-build #873 (See https://builds.apache.org/job/Tajo-master-build/873/ ) TAJO-1846 : Python temp directory path should be selected differently based on user platform. (jihoonson: rev 8ef49f0186ce813d44ba997b0fdde91497c87cda) CHANGES tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #515 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/515/)
        TAJO-1846: Python temp directory path should be selected differently based on user platform. (jihoonson: rev 8ef49f0186ce813d44ba997b0fdde91497c87cda)

        • tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java
        • CHANGES
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #515 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/515/ ) TAJO-1846 : Python temp directory path should be selected differently based on user platform. (jihoonson: rev 8ef49f0186ce813d44ba997b0fdde91497c87cda) tajo-plan/src/main/java/org/apache/tajo/plan/function/python/PythonScriptEngine.java CHANGES
        Hide
        jihoonson Jihoon Son added a comment -

        Thank you for your contribution.
        This patch is committed to master and 0.11.

        Show
        jihoonson Jihoon Son added a comment - Thank you for your contribution. This patch is committed to master and 0.11.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/761

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/761
        Hide
        jihoonson Jihoon Son added a comment -

        I'll change the issue title to 'Python temp directory path should be selected differently based on user platform'.

        Show
        jihoonson Jihoon Son added a comment - I'll change the issue title to 'Python temp directory path should be selected differently based on user platform'.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/761#issuecomment-140949149

        +1 I'll commit shortly.

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/761#issuecomment-140949149 +1 I'll commit shortly.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user dkhwangbo opened a pull request:

        https://github.com/apache/tajo/pull/761

        TAJO-1846: Write the directory address abide by user's OS

        in PythonScriptEngine, Some of variable has no consideration in user's os.
        Cannot run start-tajo.cmd in Windows 7 because of this problem.

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/dkhwangbo/tajo TAJO-1846

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/761.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 #761


        commit 96952d52950fde2a6b84661794838d68dab8b792
        Author: Dongkyu Hwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-07T07:53:33Z

        reset

        commit 04f240b5aa594c35084a081442284da96362005f
        Author: Dongkyu Hwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-07T08:00:43Z

        reset

        commit 13fbe052d66af951b030188f3602e1bf5cfa7ebb
        Author: Dongkyu Hwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-08T00:58:34Z

        Merge remote-tracking branch 'upstream/master'

        commit 0b5c1de2fe17af4fb3c8d3e774c20efd74c753f9
        Author: Dongkyu Hwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-08T04:48:46Z

        Merge remote-tracking branch 'upstream/master'

        commit 98492d8507760c13de45b2923c98ed7f485bda32
        Author: Dongkyu Hwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-09T01:09:59Z

        Merge remote-tracking branch 'upstream/master'

        commit 976c875a17c46084f98db6d861992752a9773598
        Author: dkhwangbo <dkhwangbo@gruter.com>
        Date: 2015-09-09T08:39:54Z

        Merge remote-tracking branch 'upstream/master'

        commit 4099259b0996e9358c2dffc4fa49dd12f2b799c8
        Author: dkhwangbo <hwangbodk@gmail.com>
        Date: 2015-09-10T00:59:55Z

        Merge remote-tracking branch 'upstream/master'

        commit d4a85b5ab076d42eb7c162efad4c3185abd4f205
        Author: dkhwangbo <hwangbodk@gmail.com>
        Date: 2015-09-11T01:36:25Z

        Merge remote-tracking branch 'upstream/master'

        commit 3ec01165412047e4d24403473a714109ec1ca426
        Author: dkhwangbo <hwangbodk@gmail.com>
        Date: 2015-09-11T09:52:29Z

        Merge remote-tracking branch 'upstream/master'

        commit 87c3ccef2f2b025cdf3c3d172dfae8c55f69eba2
        Author: dkhwangbo <hwangbodk@gmail.com>
        Date: 2015-09-14T01:37:33Z

        Merge remote-tracking branch 'upstream/master'

        commit b37acd773b72f251b923d177d469139ef88ab230
        Author: dkhwangbo <hwangbodk@gmail.com>
        Date: 2015-09-14T06:36:38Z

        Merge remote-tracking branch 'upstream/master'

        commit 61a2c8ab985bb95f59b611a74e6d14ea630463b9
        Author: Dongkyu Hwangbo <hwangbodk@gmail.com>
        Date: 2015-09-16T06:58:13Z

        resolve conflict

        commit 0a6e37c1ce940cc1cb60b3c31ed5785a5255f52b
        Author: Dongkyu Hwangbo <hwangbodk@gmail.com>
        Date: 2015-09-16T07:02:57Z

        rollback unnecessarily changed

        commit 8dd4b0f7e295184f0a39cd88a3f9097e9f52cc9a
        Author: Dongkyu Hwangbo <hwangbodk@gmail.com>
        Date: 2015-09-16T07:06:55Z

        rollback unnecessarily changed


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user dkhwangbo opened a pull request: https://github.com/apache/tajo/pull/761 TAJO-1846 : Write the directory address abide by user's OS in PythonScriptEngine, Some of variable has no consideration in user's os. Cannot run start-tajo.cmd in Windows 7 because of this problem. You can merge this pull request into a Git repository by running: $ git pull https://github.com/dkhwangbo/tajo TAJO-1846 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/761.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 #761 commit 96952d52950fde2a6b84661794838d68dab8b792 Author: Dongkyu Hwangbo <dkhwangbo@gruter.com> Date: 2015-09-07T07:53:33Z reset commit 04f240b5aa594c35084a081442284da96362005f Author: Dongkyu Hwangbo <dkhwangbo@gruter.com> Date: 2015-09-07T08:00:43Z reset commit 13fbe052d66af951b030188f3602e1bf5cfa7ebb Author: Dongkyu Hwangbo <dkhwangbo@gruter.com> Date: 2015-09-08T00:58:34Z Merge remote-tracking branch 'upstream/master' commit 0b5c1de2fe17af4fb3c8d3e774c20efd74c753f9 Author: Dongkyu Hwangbo <dkhwangbo@gruter.com> Date: 2015-09-08T04:48:46Z Merge remote-tracking branch 'upstream/master' commit 98492d8507760c13de45b2923c98ed7f485bda32 Author: Dongkyu Hwangbo <dkhwangbo@gruter.com> Date: 2015-09-09T01:09:59Z Merge remote-tracking branch 'upstream/master' commit 976c875a17c46084f98db6d861992752a9773598 Author: dkhwangbo <dkhwangbo@gruter.com> Date: 2015-09-09T08:39:54Z Merge remote-tracking branch 'upstream/master' commit 4099259b0996e9358c2dffc4fa49dd12f2b799c8 Author: dkhwangbo <hwangbodk@gmail.com> Date: 2015-09-10T00:59:55Z Merge remote-tracking branch 'upstream/master' commit d4a85b5ab076d42eb7c162efad4c3185abd4f205 Author: dkhwangbo <hwangbodk@gmail.com> Date: 2015-09-11T01:36:25Z Merge remote-tracking branch 'upstream/master' commit 3ec01165412047e4d24403473a714109ec1ca426 Author: dkhwangbo <hwangbodk@gmail.com> Date: 2015-09-11T09:52:29Z Merge remote-tracking branch 'upstream/master' commit 87c3ccef2f2b025cdf3c3d172dfae8c55f69eba2 Author: dkhwangbo <hwangbodk@gmail.com> Date: 2015-09-14T01:37:33Z Merge remote-tracking branch 'upstream/master' commit b37acd773b72f251b923d177d469139ef88ab230 Author: dkhwangbo <hwangbodk@gmail.com> Date: 2015-09-14T06:36:38Z Merge remote-tracking branch 'upstream/master' commit 61a2c8ab985bb95f59b611a74e6d14ea630463b9 Author: Dongkyu Hwangbo <hwangbodk@gmail.com> Date: 2015-09-16T06:58:13Z resolve conflict commit 0a6e37c1ce940cc1cb60b3c31ed5785a5255f52b Author: Dongkyu Hwangbo <hwangbodk@gmail.com> Date: 2015-09-16T07:02:57Z rollback unnecessarily changed commit 8dd4b0f7e295184f0a39cd88a3f9097e9f52cc9a Author: Dongkyu Hwangbo <hwangbodk@gmail.com> Date: 2015-09-16T07:06:55Z rollback unnecessarily changed
        Hide
        dkhwangbo Dongkyu Hwangbo added a comment -

        I renew summary and description of this subject.

        Show
        dkhwangbo Dongkyu Hwangbo added a comment - I renew summary and description of this subject.

          People

          • Assignee:
            dkhwangbo Dongkyu Hwangbo
            Reporter:
            dkhwangbo Dongkyu Hwangbo
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development