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

When a task is failed, ParquetAppender::close causes NPE.

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None

      Description

      See the title. The below is error stack trace.

      2014-12-04 15:22:07,949 ERROR org.apache.tajo.worker.TaskRunner:
      java.lang.NullPointerException
      at org.apache.tajo.storage.parquet.ParquetAppender.close(ParquetAppender.java:130)
      at org.apache.tajo.engine.planner.physical.StoreTableExec.close(StoreTableExec.java:137)
      at org.apache.tajo.worker.Task.run(Task.java:458)
      at org.apache.tajo.worker.TaskRunner$1.run(TaskRunner.java:276)
      at java.lang.Thread.run(Thread.java:745)
      02/RESULT/part-01-000003-000
      

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user eminency closed the pull request at:

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

          Show
          githubbot ASF GitHub Bot added a comment - Github user eminency closed the pull request at: https://github.com/apache/tajo/pull/309
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user eminency opened a pull request:

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

          Tajo-1258: Close() derived FileAppender should consider inited state

          Close() for FileAppender is added and derived classes use it to check initialization state.

          This modification is also including resolution for TAJO-1227.

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

          $ git pull https://github.com/eminency/tajo TAJO-1258

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

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


          commit 205331847c2a44381299ab1c9fed4c9c7370472e
          Author: Jongyoung Park <eminency@gmail.com>
          Date: 2014-12-18T16:19:29Z

          Close() for FileAppender is added and derived classes are modified

          commit 7fb632de1860065f287b87498abe9a85eb95848c
          Author: Jihoon Son <jihoonson@apache.org>
          Date: 2014-12-18T07:50:09Z

          TAJO-1232: Implicit groupby queries with LIMIT lead to wrong results. (jihoon)


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user eminency opened a pull request: https://github.com/apache/tajo/pull/309 Tajo-1258: Close() derived FileAppender should consider inited state Close() for FileAppender is added and derived classes use it to check initialization state. This modification is also including resolution for TAJO-1227 . You can merge this pull request into a Git repository by running: $ git pull https://github.com/eminency/tajo TAJO-1258 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/309.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 #309 commit 205331847c2a44381299ab1c9fed4c9c7370472e Author: Jongyoung Park <eminency@gmail.com> Date: 2014-12-18T16:19:29Z Close() for FileAppender is added and derived classes are modified commit 7fb632de1860065f287b87498abe9a85eb95848c Author: Jihoon Son <jihoonson@apache.org> Date: 2014-12-18T07:50:09Z TAJO-1232 : Implicit groupby queries with LIMIT lead to wrong results. (jihoon)

            People

            • Assignee:
              Unassigned
              Reporter:
              hyunsik Hyunsik Choi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development