Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/22683
  
    I believe most of the Spark code really uses units like KiB, multiples of 
1024, as you see here. However it's often referred to as KB, incorrectly, and 
so on. It seems like the UI is the only place that uses 1000 and KB, which is 
consistent in itself, but should really be 1024/KiB. that change to the UI 
seems fine. I think there are a few comments and docs in the code that really 
also should change to say KiB and so on, and we can make those changes here, 
but it's OK to focus on making the UI based on KiB.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to