Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8019239

Properly clean up sjavac heuristics

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P4
    • Resolution: Not an Issue
    • Affects Version/s: 8
    • Fix Version/s: 9
    • Component/s: tools
    • Labels:

      Description

      Sjavac currently has heuristics to used to figure out how many threads and how many classes can be compiled at a time based on size of heap and if pointers are 32 or 64 bit. The problem is, this is currently happening in the client process and not the server process, so the information gathered might be wrong.

      Also, the heuristics are very crude when reaching the conclusion that the amount of memory isn't enough for compiling everything in one batch. This makes compilation very slow if you happen to fall under the treshold.

        Issue Links

          Activity

          Hide
          alundblad Andreas Lundblad (Inactive) added a comment -
          Looking at the code, the first part (regarding architecture / available memory on client side rather than server side) seems to have been resolved.

          Will start looking into the second part now.
          Show
          alundblad Andreas Lundblad (Inactive) added a comment - Looking at the code, the first part (regarding architecture / available memory on client side rather than server side) seems to have been resolved. Will start looking into the second part now.
          Hide
          alundblad Andreas Lundblad (Inactive) added a comment -
          This is no longer an issue in JDK 9 since the build has been split up into multiple considerable smaller chunks.
          Show
          alundblad Andreas Lundblad (Inactive) added a comment - This is no longer an issue in JDK 9 since the build has been split up into multiple considerable smaller chunks.

            People

            • Assignee:
              alundblad Andreas Lundblad (Inactive)
              Reporter:
              erikj Erik Joelsson
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: