Improve Decimal128::logarithm functions

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 8.2.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Server Programmability
    • Fully Compatible
    • Programmability 2025-06-09
    • None
    • 3
    • TBD
    • None
    • None
    • None
    • None
    • None
    • None
    • 0

      We should get rid of the logarithm taking a base. Just have 3 functions. log (implicitly base e), log10, and log2. That's all the bid128 library gives us anyway. and these are names which match std:: terminology.
      No need to get clever about it. Any other bases (very uncommon) would be accomplished by dividing logs. That's exactly what it does internally anyway. Why does naturalLogarithm take signalingFlags but the wrapper logarithm doesn't? That's also inconsistent.

      This is like SERVER-91935, and addresses the same API clarity problem but with a different solution.

            Assignee:
            Billy Donahue
            Reporter:
            Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: