Uploaded image for project: 'Infinispan'
  1. Infinispan
  2. ISPN-7802

Use chunked reads/writes in TcpTransport

    XMLWordPrintable

Details

    Description

      The buffering implementation of TcpTransport.socketInputStream needs to use fixed size input buffer, rather than growing as BufferedInputStream does. Growing buffer can lead to excessive memory consumption on heap, and more importantly, by direct memory allocated in JDK classes and pooled in thread-local caches.

      EDIT: As Tristan pointed out, the BufferedInputStream does not grow in our use case, but if TcpTransport passes a big byte[] buffer, the implementation works as read through/write through. Therefore we need to chunk all calls.

      Attachments

        Issue Links

          is cloned by

          Bug - A problem that impairs or prevents the functions of the product. JDG-973 Use non-growing buffered input stream in TcpTransport

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed

          Bug - A problem that impairs or prevents the functions of the product. JDG-974 Use non-growing buffered input stream in TcpTransport

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed
          relates to

          Bug - A problem that impairs or prevents the functions of the product. ISPN-5740 HotRod client write buffer is too large

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed

          Activity

            People

              ttarrant@redhat.com Tristan Tarrant
              rvansa1@redhat.com Radim Vansa (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: