[jira] [Commented] (THRIFT-4335) Golang TPipe Windows implementation

["Sylwester Lachiewicz (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/THRIFT-4335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18115798#comment-18115798 ] 

Sylwester Lachiewicz commented on THRIFT-4335:
----------------------------------------------

Status on master: lib/go has no pipe transport, and the library still has no third-party dependency, so adding go-winio would be the first one. The license is not the obstacle, MIT is Category A. A caller does not need the library to change, though: [NewTSocketFromConnTimeout|https://github.com/apache/thrift/blob/master/lib/go/thrift/socket.go] accepts any {{net.Conn}}, so a go-winio pipe connection can be wrapped in a TSocket today and used with every protocol. If that covers the use case, this can be closed.

_This comment was created with AI assistance._

> Golang TPipe Windows implementation
> -----------------------------------
>
>                 Key: THRIFT-4335
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4335
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Go - Library
>            Reporter: Zach Wasserman
>            Priority: Major
>
> We are interested in adding support for the TPipe transport in the Golang Windows library. This support already exists in the C++ library (https://thrift.apache.org/lib/cpp#named-pipes), and is useful for an alternative to TCP on Windows. Would the maintainers be interested in merging a patch adding this functionality?
> Microsoft publishes an (MIT licensed) library that provides {{net.Conn}} support for named pipes (https://github.com/Microsoft/go-winio/blob/master/pipe.go#L138, https://github.com/Microsoft/go-winio/blob/master/pipe.go#L317). Is it possible to add this as a dependency for the Thrift Go library?
> Any guidance would be appreciated, as I'd love to start implementing it ASAP.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)