Skip to content

[16.0][OU-IMP] account: performance#5357

Merged
pedrobaeza merged 1 commit intoOCA:16.0from
FernandoRomera:16.0-fromera-openupgrade_scripts
Sep 3, 2025
Merged

[16.0][OU-IMP] account: performance#5357
pedrobaeza merged 1 commit intoOCA:16.0from
FernandoRomera:16.0-fromera-openupgrade_scripts

Conversation

@FernandoRomera
Copy link
Copy Markdown
Contributor

@FernandoRomera FernandoRomera commented Sep 2, 2025

Before

2025-08-28 10:03:05,845 59373 DEBUG demo OpenUpgrade: 4057774 rows affected after 1:02:28.371871 running 

"QUERY PLAN"
"Update on account_move_line aml  (cost=0.85..609279.24 rows=0 width=0)"
"  ->  Nested Loop Left Join  (cost=0.85..609279.24 rows=1 width=56)"
"        Join Filter: (aa.id = aml_1.account_id)"
"        ->  Nested Loop  (cost=0.85..606388.31 rows=1 width=34)"
"              ->  Nested Loop  (cost=0.43..606379.87 rows=1 width=24)"
"                    ->  Seq Scan on account_move_line aml_1  (cost=0.00..606371.42 rows=1 width=22)"
"                          Filter: (display_type IS NULL)"
"                    ->  Index Scan using account_move_line_pkey on account_move_line aml  (cost=0.43..8.45 rows=1 width=10)"
"                          Index Cond: (id = aml_1.id)"
"              ->  Index Scan using account_move_pkey on account_move am  (cost=0.42..8.44 rows=1 width=18)"
"                    Index Cond: (id = aml_1.move_id)"
"        ->  Seq Scan on account_account aa  (cost=0.00..2436.52 rows=36352 width=23)"
"JIT:"
"  Functions: 18"
"  Options: Inlining true, Optimization true, Expressions true, Deforming true"

After

2025-08-29 09:39:20,554 82222 DEBUG demo OpenUpgrade: 4057774 rows affected after 0:17:19.349663 running 

"QUERY PLAN"
"Update on account_move_line aml2  (cost=0.00..606396.63 rows=0 width=0)"
"  ->  Seq Scan on account_move_line aml2  (cost=0.00..606396.63 rows=1 width=38)"
"        Filter: (display_type IS NULL)"
"        SubPlan 1"
"          ->  Nested Loop  (cost=1.15..25.21 rows=1 width=32)"
"                ->  Nested Loop  (cost=0.85..16.89 rows=1 width=16)"
"                      ->  Index Scan using account_move_line_pkey on account_move_line aml  (cost=0.43..8.45 rows=1 width=12)"
"                            Index Cond: (id = aml2.id)"
"                      ->  Index Scan using account_move_pkey on account_move am  (cost=0.42..8.44 rows=1 width=12)"
"                            Index Cond: (id = aml.move_id)"
"                ->  Index Scan using account_account_pkey on account_account aa  (cost=0.29..8.31 rows=1 width=17)"
"                      Index Cond: (id = aml.account_id)"
"JIT:"
"  Functions: 18"
"  Options: Inlining true, Optimization true, Expressions true, Deforming true"

@pedrobaeza pedrobaeza added this to the 16.0 milestone Sep 2, 2025
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the commit message, please remove [16.0] and put [OU-IMP] as tag.

Comment thread openupgrade_scripts/scripts/account/16.0.1.2/pre-migration.py Outdated
@FernandoRomera FernandoRomera force-pushed the 16.0-fromera-openupgrade_scripts branch 3 times, most recently from 41375d6 to 3ab8b36 Compare September 3, 2025 05:37
@FernandoRomera FernandoRomera changed the title [16.0][IMP] account: performace [OU-IMP] account: performace Sep 3, 2025
@FernandoRomera FernandoRomera changed the title [OU-IMP] account: performace [OU-IMP] account: performance Sep 3, 2025
@pedrobaeza pedrobaeza changed the title [OU-IMP] account: performance [16.0][OU-IMP] account: performance Sep 3, 2025
Comment thread openupgrade_scripts/scripts/account/16.0.1.2/pre-migration.py Outdated
@FernandoRomera FernandoRomera force-pushed the 16.0-fromera-openupgrade_scripts branch from 3ab8b36 to bf97344 Compare September 3, 2025 08:17
@FernandoRomera FernandoRomera force-pushed the 16.0-fromera-openupgrade_scripts branch from bf97344 to 92d4387 Compare September 3, 2025 08:19
@pedrobaeza pedrobaeza merged commit ec09c12 into OCA:16.0 Sep 3, 2025
2 checks passed
@FernandoRomera FernandoRomera deleted the 16.0-fromera-openupgrade_scripts branch September 3, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants