Uploaded image for project: 'Parquet'
  1. Parquet
  2. PARQUET-66

InternalParquetRecordWriter int overflow causes unnecessary memory check warning

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.0
    • Fix Version/s: 1.6.0
    • Component/s: parquet-mr
    • Labels:
      None

      Description

      flushStore() contains a check on the amount of memory allocated and if it is > 3 * blockSize it logs a warning. If blockSize is large enough, e.g. 1GiB then 3 * blockSize will overflow and be negative causing the warning to always be logged.

      if (store.allocatedSize() > 3 * blockSize) {
          LOG.warn("Too much memory used: " + store.memUsageString());
      }
      

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              mrhariseldon Eric Snyder
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: