Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-5463

Make RamUsageEstimator.(human)sizeOf(Object) a forbidden API

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.8
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      We have had a few issues with RamUsageEstimator recently so I think we should consider making the sizeOf(Object) and humanSizeOf(Object) methods forbidden under src/java (however still allowed for tests as it is handy to check the size computations which are done "manually"). However, sizeOf(byte[]), shallowSizeOf(Class), etc. remain useful so I think we should keep them allowed.

        Attachments

        1. LUCENE-5463.patch
          16 kB
          Adrien Grand
        2. LUCENE-5463.patch
          3 kB
          Robert Muir
        3. LUCENE-5463-2.patch
          10 kB
          Adrien Grand
        4. LUCENE-5463-2.patch
          9 kB
          Adrien Grand

          Activity

            People

            • Assignee:
              jpountz Adrien Grand
              Reporter:
              jpountz Adrien Grand
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: