-
Notifications
You must be signed in to change notification settings - Fork 197
change AOCO_Compression test case to validate pg_relation_size and get_ao_compression_ratio within ±10% expected range #493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@tuhaihe Dianjin, it appears pipeline failed before ICW, could you please help? |
|
Hey, @congxuebin: had retriggered the pipelines again. |
|
@tuhaihe Hi Dianjin, thanks for retriggering it. But it failed again at check out step. |
|
@tuhaihe Dianjin, it appears the git runner issue has resolved, could you please re-trigger this pipeline? Thanks in advance. |
|
The test case that I modified has successfully executed via the pipeline. But it failed on something else. AOCO_Compression ... ok 28984 ms (diff 219 ms) Could you please help re-trigger the ICW? or actually it should be ok to move forward instead if is possible. |
|
Hi Max @my-ship-it , The test case I modified ran successfully via this pipeline, but the pipeline failed due to something else. I don't think it's worth another rerun for this. AOCO_Compression ... ok 28984 ms (diff 219 ms) I heard reviewer has super authority to directly merge code change. Would you please review it, if approved, proceed with the merge? |
|
@congxuebin: I have help retrigger the pipelines again. Don't worry, the pr will be merged only after passing all the tests. |
|
Thanks. All checks have passed. |
|
@congxuebin Please don't use Merge branch |
…t_ao_compression_ratio within ±10% expected range according to issue apache#487
|
@tuhaihe Hi Dianjin, my test case succeeded. AOCO_Compression ... ok 26721 ms (diff 219 ms) Would you please help to re-trigger the pipeline as it failed on something else? Thanks. |
|
@congxuebin Sure, re-triggered the test again. If still failed, plz let me know. |
change AOCO_Compression test case to validate pg_relation_size and get_ao_compression_ratio within ±10% expected range according to issue apache#487 (apache#493) Co-authored-by: congxuebin <[email protected]> (cherry picked from commit be38ff5)
fix #487
In the UOS 1050a aarch64 operating system, there are slight differences in the storage space occupied by table data and the compression ratio compared to other systems. Therefore, modify test cases to validate pg_relation_size and get_ao_compression_ratio within a ±10% expected range.
Outlined below are two modifications.