Skip to content

[MINOR][UI] Increasing the number of significant digits for Fraction Cached of RDD#42373

Closed
yaooqinn wants to merge 2 commits intoapache:masterfrom
yaooqinn:uiminor
Closed

[MINOR][UI] Increasing the number of significant digits for Fraction Cached of RDD#42373
yaooqinn wants to merge 2 commits intoapache:masterfrom
yaooqinn:uiminor

Conversation

@yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Aug 7, 2023

What changes were proposed in this pull request?

This PR is a typo improvement for increasing the number of significant digits for Fraction Cached of RDD that shows on the Storage Tab.

Why are the changes needed?

improves accuracy and precision

image

Does this PR introduce any user-facing change?

Yes, the Fraction Cached on Storage Page increases the fractional length from 0 to 2

How was this patch tested?

locally verified

@yaooqinn
Copy link
Member Author

yaooqinn commented Aug 7, 2023

cc @sarutak @HyukjinKwon thanks

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.5.

HyukjinKwon pushed a commit that referenced this pull request Aug 7, 2023
…Cached of RDD

### What changes were proposed in this pull request?

This PR is a typo improvement for increasing the number of significant digits for Fraction Cached of RDD that shows on the Storage Tab.

### Why are the changes needed?

improves accuracy and precision

![image](https://github.com/apache/spark/assets/8326978/7106352c-b806-4953-8938-c4cba8ea1191)

### Does this PR introduce _any_ user-facing change?

Yes, the Fraction Cached on Storage Page increases the fractional length from 0 to 2

### How was this patch tested?

locally verified

Closes #42373 from yaooqinn/uiminor.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit f47a256)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@dongjoon-hyun
Copy link
Member

Thank you. Is this Apache Spark 3.5-only bug fix?

@HyukjinKwon
Copy link
Member

Oh I just cherry-picked this because it seems fairly minor but I don't mind reverting this out of branch-3.5

@yaooqinn yaooqinn deleted the uiminor branch August 8, 2023 01:45
valentinp17 pushed a commit to valentinp17/spark that referenced this pull request Aug 24, 2023
…Cached of RDD

### What changes were proposed in this pull request?

This PR is a typo improvement for increasing the number of significant digits for Fraction Cached of RDD that shows on the Storage Tab.

### Why are the changes needed?

improves accuracy and precision

![image](https://github.com/apache/spark/assets/8326978/7106352c-b806-4953-8938-c4cba8ea1191)

### Does this PR introduce _any_ user-facing change?

Yes, the Fraction Cached on Storage Page increases the fractional length from 0 to 2

### How was this patch tested?

locally verified

Closes apache#42373 from yaooqinn/uiminor.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments