[jira] [Commented] (THRIFT-3292) Implement TZlibTransport in Go
[ https://issues.apache.org/jira/browse/THRIFT-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14660881#comment-14660881 ] ASF GitHub Bot commented on THRIFT-3292: Github user Jens-G commented on the pull request: https://github.com/apache/thrift/pull/580#issuecomment-128523983 Could you please integrate the new Transport into [the test suite](https://github.com/apache/thrift/tree/master/test/go)? Implement TZlibTransport in Go -- Key: THRIFT-3292 URL: https://issues.apache.org/jira/browse/THRIFT-3292 Project: Thrift Issue Type: New Feature Components: Go - Library Reporter: Paul Magrath Assignee: Paul Magrath Priority: Minor TZlibTransport compatible with Java version of TZlibTransport -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] thrift pull request: THRIFT-3292 Add a Zlib transport for Go
Github user Jens-G commented on the pull request: https://github.com/apache/thrift/pull/580#issuecomment-128523983 Could you please integrate the new Transport into [the test suite](https://github.com/apache/thrift/tree/master/test/go)? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[jira] [Updated] (THRIFT-3292) Implement TZlibTransport in Go
[ https://issues.apache.org/jira/browse/THRIFT-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Geyer updated THRIFT-3292: --- Assignee: Paul Magrath (was: Jens Geyer) Implement TZlibTransport in Go -- Key: THRIFT-3292 URL: https://issues.apache.org/jira/browse/THRIFT-3292 Project: Thrift Issue Type: New Feature Components: Go - Library Reporter: Paul Magrath Assignee: Paul Magrath Priority: Minor TZlibTransport compatible with Java version of TZlibTransport -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Assigned] (THRIFT-3292) Implement TZlibTransport in Go
[ https://issues.apache.org/jira/browse/THRIFT-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Geyer reassigned THRIFT-3292: -- Assignee: Jens Geyer Implement TZlibTransport in Go -- Key: THRIFT-3292 URL: https://issues.apache.org/jira/browse/THRIFT-3292 Project: Thrift Issue Type: New Feature Components: Go - Library Reporter: Paul Magrath Assignee: Jens Geyer Priority: Minor TZlibTransport compatible with Java version of TZlibTransport -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (THRIFT-3292) Implement TZlibTransport in Go
[ https://issues.apache.org/jira/browse/THRIFT-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14660416#comment-14660416 ] ASF GitHub Bot commented on THRIFT-3292: GitHub user Magrath opened a pull request: https://github.com/apache/thrift/pull/580 THRIFT-3292 Add a Zlib transport for Go This is a TZlibTransport implementation compatible with the Java version of TZlibTransport, at least from my (admittedly limited) testing. https://issues.apache.org/jira/browse/THRIFT-3292 You can merge this pull request into a Git repository by running: $ git pull https://github.com/Magrath/thrift THRIFT-3292 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/580.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 #580 commit 46ce50519446723197ce306186752345e6f21b52 Author: Paul Magrath p...@swiftkey.com Date: 2015-08-06T14:30:48Z THRIFT-3292 Add a Zlib transport for Go Implement TZlibTransport in Go -- Key: THRIFT-3292 URL: https://issues.apache.org/jira/browse/THRIFT-3292 Project: Thrift Issue Type: New Feature Components: Go - Library Reporter: Paul Magrath Priority: Minor TZlibTransport compatible with Java version of TZlibTransport -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] thrift pull request: THRIFT-3292 Add a Zlib transport for Go
GitHub user Magrath opened a pull request: https://github.com/apache/thrift/pull/580 THRIFT-3292 Add a Zlib transport for Go This is a TZlibTransport implementation compatible with the Java version of TZlibTransport, at least from my (admittedly limited) testing. https://issues.apache.org/jira/browse/THRIFT-3292 You can merge this pull request into a Git repository by running: $ git pull https://github.com/Magrath/thrift THRIFT-3292 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/580.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 #580 commit 46ce50519446723197ce306186752345e6f21b52 Author: Paul Magrath p...@swiftkey.com Date: 2015-08-06T14:30:48Z THRIFT-3292 Add a Zlib transport for Go --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[jira] [Created] (THRIFT-3292) Implement TZlibTransport in Go
Paul Magrath created THRIFT-3292: Summary: Implement TZlibTransport in Go Key: THRIFT-3292 URL: https://issues.apache.org/jira/browse/THRIFT-3292 Project: Thrift Issue Type: New Feature Components: Go - Library Reporter: Paul Magrath Priority: Minor TZlibTransport compatible with Java version of TZlibTransport -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (THRIFT-2199) Remove Dense protocol (was: move to Contrib)
[ https://issues.apache.org/jira/browse/THRIFT-2199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14659674#comment-14659674 ] Randy Abernethy commented on THRIFT-2199: - thanks Jens. Remove Dense protocol (was: move to Contrib) Key: THRIFT-2199 URL: https://issues.apache.org/jira/browse/THRIFT-2199 Project: Thrift Issue Type: Improvement Components: Compiler (General) Affects Versions: 1.0 Environment: All Reporter: Randy Abernethy Assignee: Randy Abernethy Priority: Minor Fix For: 0.9.3 Attachments: 0001-dense-removal.patch In response to recent emails I suggest we move the Dense protocol to contrib. It is in an experimental state (per comments) and has not been enhanced for over three years. It impacts complexity of Thrift in a fairly broad fashion, having bearing on compilation and compiler output even when not used. If no one disagrees with this action I would be happy to put a patch together to relocate dense to contrib. Would fix THRIFT-2200 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (THRIFT-2199) Remove Dense protocol (was: move to Contrib)
[ https://issues.apache.org/jira/browse/THRIFT-2199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14659725#comment-14659725 ] Hudson commented on THRIFT-2199: SUCCESS: Integrated in Thrift #1639 (See [https://builds.apache.org/job/Thrift/1639/]) THRIFT-2199:Remove dense protocol (ra: rev 8dbe5f60a7c1401302e10aec41069f9c385d34a1) * lib/cpp/src/thrift/TReflectionLocal.h * compiler/cpp/src/generate/t_cpp_generator.cc * lib/cpp/src/thrift/protocol/TDenseProtocol.h * lib/cpp/test/Makefile.am * compiler/cpp/src/parse/t_list.h * compiler/cpp/src/parse/parse.cc * lib/cpp/libthrift.vcxproj * lib/cpp/libthrift.vcxproj.filters * compiler/cpp/src/parse/t_struct.h * compiler/cpp/src/parse/t_typedef.h * lib/cpp/Makefile.am * compiler/cpp/src/main.cc * compiler/cpp/src/generate/t_json_generator.cc * lib/cpp/src/thrift/protocol/TDenseProtocol.cpp * compiler/cpp/src/parse/t_base_type.h * compiler/cpp/src/parse/t_service.h * lib/cpp/src/thrift/Thrift.h * compiler/cpp/src/parse/t_set.h * compiler/cpp/src/parse/t_type.h * compiler/cpp/src/parse/t_field.h * compiler/cpp/src/parse/t_map.h * lib/cpp/test/DenseProtoTest.cpp * lib/cpp/test/CMakeLists.txt * lib/cpp/CMakeLists.txt * lib/cpp/src/thrift/protocol/TBinaryProtocol.h Remove Dense protocol (was: move to Contrib) Key: THRIFT-2199 URL: https://issues.apache.org/jira/browse/THRIFT-2199 Project: Thrift Issue Type: Improvement Components: Compiler (General) Affects Versions: 1.0 Environment: All Reporter: Randy Abernethy Assignee: Randy Abernethy Priority: Minor Fix For: 0.9.3 Attachments: 0001-dense-removal.patch In response to recent emails I suggest we move the Dense protocol to contrib. It is in an experimental state (per comments) and has not been enhanced for over three years. It impacts complexity of Thrift in a fairly broad fashion, having bearing on compilation and compiler output even when not used. If no one disagrees with this action I would be happy to put a patch together to relocate dense to contrib. Would fix THRIFT-2200 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Created] (THRIFT-3291) Java dependencies missing from tutorial
Jens Geyer created THRIFT-3291: -- Summary: Java dependencies missing from tutorial Key: THRIFT-3291 URL: https://issues.apache.org/jira/browse/THRIFT-3291 Project: Thrift Issue Type: Bug Components: Website Reporter: Jens Geyer Priority: Minor The dependencies for Java are not listed in the appropriate section. [Particularly, slf4j is not mentioned|http://stackoverflow.com/q/31848019/499466]. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Resolved] (THRIFT-2199) Remove Dense protocol (was: move to Contrib)
[ https://issues.apache.org/jira/browse/THRIFT-2199?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Randy Abernethy resolved THRIFT-2199. - Resolution: Fixed committed Remove Dense protocol (was: move to Contrib) Key: THRIFT-2199 URL: https://issues.apache.org/jira/browse/THRIFT-2199 Project: Thrift Issue Type: Improvement Components: Compiler (General) Affects Versions: 1.0 Environment: All Reporter: Randy Abernethy Assignee: Randy Abernethy Priority: Minor Fix For: 0.9.3 Attachments: 0001-dense-removal.patch In response to recent emails I suggest we move the Dense protocol to contrib. It is in an experimental state (per comments) and has not been enhanced for over three years. It impacts complexity of Thrift in a fairly broad fashion, having bearing on compilation and compiler output even when not used. If no one disagrees with this action I would be happy to put a patch together to relocate dense to contrib. Would fix THRIFT-2200 -- This message was sent by Atlassian JIRA (v6.3.4#6332)