Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-4839

Remove embedded buffering/framed options from TCP transports

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 0.13.0
    • netstd - Library
    • None

    Description

      The current TServerSocketTransport and TTlsServerSocketTransport classes are embedding framed and buffered options instead of using the existing transport factories. This is causing duplicate code when there need not be, and it doesn't make sense to bring all transports into alignment.

      Attachments

        Issue Links

        supercedes

        Improvement - An improvement or enhancement to an existing feature or task. THRIFT-4828 Add framed/buffered options to TNamedPipeServerTransport

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Closed
        links to

        Web Link GitHub Pull Request #1770

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            animusblue Kyle Smith
            animusblue Kyle Smith
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 20m
                20m

                Slack

                  Issue deployment