[ORC-975] Avoid double counting closestFixedBits in percentileBits method

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.8.0
• Fix Version/s: 1.8.0
• Component/s: Java
• Assignee: Yiqun Zhang
• Reporter: Yiqun Zhang
• Created: 28/Aug/21 09:06
• Updated: 03/Sep/22 22:43
• Resolved: 30/Aug/21 04:02

Description

SerializationUtils

310  int idx = encodeBitWidth(findClosestNumBits(data[i]));

public int encodeBitWidth(int n) {
    n = getClosestFixedBits(n);
    .....
}

public int findClosestNumBits(long value) {
  final int numBits = 64 - Long.numberOfLeadingZeros(value);
  return getClosestFixedBits(numBits);
}

getClosestFixedBits is called twice.

Issue Links

• links to: GitHub Pull Request #886