0050549: Cycles in entities that represents data as a tree can lead to transaction that does not finish, 100% use of CPU

Openbravo Issue Tracking System - Openbravo ERP

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0050549

Openbravo ERP

A. Platform

public

2022-10-17 11:16

2022-10-31 16:41

Reporter

AugustoMauch

Assigned To

AugustoMauch

Priority

normal

Severity

critical

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

PR23Q1

Merge Request Status

Review Assigned To

OBNetwork customer

Web browser

Modules

Core

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0050549: Cycles in entities that represents data as a tree can lead to transaction that does not finish, 100% use of CPU

Description

The data of some entities (i.e. ProductCategory) can be represented as a tree, where each entry has a single parent entry. These entities can be visualized in a tree grid.

If the data of any of these entities is not proper and there are cycles (i.e. the parent of an entry is the entry itself), then using the filter in the tree grid can lead to a database transaction that will never be closed and a close to 100% use of the CPU.

Steps To Reproduce

- Force a cycle in the hierarchy of any Openbravo entity, i.e. this will create a cycle in the product characteristic tree of the White Valley sampledata client (electronics category will be its own parent):

update ad_treenode set parent_id = '3ECCEC0B766E4A25A697187A58B15B71' where node_id = '3ECCEC0B766E4A25A697187A58B15B71';

- After that, open the Product Characteristic window under that client, change to the tree grid view and filter by entering a 'c'
- Nothing will be displayed in the grid filter and a transaction will remain open in the database. You can check it by checking pg_stat_activity and kill it by invoking "select pg_terminate_backend(pid)", being the pid the pid of the stuck transaction
- Fix the category data by executing:

update ad_treenode set parent_id = '5AAC980A56404867815CCF2B52804080' where node_id = '3ECCEC0B766E4A25A697187A58B15B71';

Proposed Solution

Additional Information

Tags

No tags attached.

Relationships

depends on	backport	0050696	PR22Q4.1	closed	AugustoMauch	Cycles in entities that represents data as a tree can lead to transaction that does not finish, 100% use of CPU
depends on	backport	0050697	PR22Q3.2	closed	AugustoMauch	Cycles in entities that represents data as a tree can lead to transaction that does not finish, 100% use of CPU
causes	defect	0050921		closed	AugustoMauch	Filter in valid tree structure results in message related to cycles in the hierarchy