[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-30 Thread blrunner
Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/1033#issuecomment-222590261 Thanks @jinossy , I rebases your comments and added more unit tests. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-30 Thread jinossy
Github user jinossy commented on a diff in the pull request: https://github.com/apache/tajo/pull/1033#discussion_r65037985 --- Diff: tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java --- @@ -104,7 +105,15 @@ public void

[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-30 Thread jinossy
Github user jinossy commented on a diff in the pull request: https://github.com/apache/tajo/pull/1033#discussion_r65037770 --- Diff: tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java --- @@ -105,6 +105,11 @@ public void

[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-30 Thread blrunner
Github user blrunner commented on a diff in the pull request: https://github.com/apache/tajo/pull/1033#discussion_r65033090 --- Diff: tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java --- @@ -105,6 +105,11 @@ public void

[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-29 Thread jinossy
Github user jinossy commented on a diff in the pull request: https://github.com/apache/tajo/pull/1033#discussion_r65017026 --- Diff: tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java --- @@ -105,6 +105,11 @@ public void

[GitHub] tajo pull request: TAJO-2164: SequenceFile print wrong values with...

2016-05-27 Thread blrunner
GitHub user blrunner opened a pull request: https://github.com/apache/tajo/pull/1033 TAJO-2164: SequenceFile print wrong values with TextSerializerDeserializer See https://issues.apache.org/jira/browse/TAJO-2164 You can merge this pull request into a Git repository by running: