You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@tajo.apache.org by jinossy <gi...@git.apache.org> on 2015/07/21 10:34:23 UTC

[GitHub] tajo pull request: TAJO-1496: Remove legacy CSVFile

GitHub user jinossy opened a pull request:

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

    TAJO-1496: Remove legacy CSVFile

    I've disable the TestTajoCli.testTimeZoneTest2. because it has a meta overwriting bug

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

    $ git pull https://github.com/jinossy/tajo TAJO-1496

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

    https://github.com/apache/tajo/pull/638.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 #638
    
----
commit d4d3263f3d64762591caf9ec942064b28475f728
Author: Jinho Kim <jh...@apache.org>
Date:   2015-07-21T06:04:26Z

    TAJO-1496

commit cc5fc6339ab928b4ebc6d5b02fefdf05ab9cebe4
Author: Jinho Kim <jh...@apache.org>
Date:   2015-07-21T06:04:35Z

    Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into TAJO-1496

commit 8d5ef7a52e62938afafbbc9636355ce81173bedc
Author: Jinho Kim <jh...@apache.org>
Date:   2015-07-21T08:31:34Z

    TAJO-1496: Remove legacy CSVFile

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1496: Remove legacy CSVFile

Posted by hyunsik <gi...@git.apache.org>.

Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/638#issuecomment-123576613
  
    #634 adds some workaround of timezone bug. So, you may need to enable the test after you commit your patch.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1496: Remove legacy CSVFile

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1496: Remove legacy CSVFile

Posted by jinossy <gi...@git.apache.org>.

Github user jinossy commented on the pull request:

    https://github.com/apache/tajo/pull/638#issuecomment-123589467
  
    I’m going to enable timezone unit test after TAJO-1343 is closed. All unit test is succeed and I will commit it


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---