nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-02-15 23:54:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf: faster count estimation (#46550)

Browse Source 
These count queries themselves take quite a long time. `estimate_count`
uses info_schema stats to guess the time.

Co-authored-by: Nabin Hait <nabinhait@gmail.com>
This commit is contained in:
Ankush Menat
2025-03-17 12:39:21 +05:30
committed by GitHub
parent 90b6e00237
commit e47a87839b
2 changed files with 3 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
erpnext/accounts/doctype/period_closing_voucher/period_closing_voucher.py
View File

@@ -139,7 +139,7 @@ class PeriodClosingVoucher(AccountsController):
self.cancel_gl_entries()
def make_gl_entries(self):
if self.get_gle_count_in_selected_period() > 5000:
if frappe.db.estimate_count("GL Entry") > 100_000:
frappe.enqueue(
process_gl_and_closing_entries,
doc=self,
@@ -154,16 +154,6 @@ class PeriodClosingVoucher(AccountsController):
else:
process_gl_and_closing_entries(self)
def get_gle_count_in_selected_period(self):
return frappe.db.count(
"GL Entry",
{
"posting_date": ["between", [self.period_start_date, self.period_end_date]],
"company": self.company,
"is_cancelled": 0,
},
)
def get_pcv_gl_entries(self):
self.pl_accounts_reverse_gle = []
self.closing_account_gle = []

15
erpnext/stock/report/batch_wise_balance_history/batch_wise_balance_history.py
View File

@@ -11,25 +11,14 @@ from erpnext.deprecation_dumpster import deprecated
from erpnext.stock.doctype.stock_closing_entry.stock_closing_entry import StockClosing
from erpnext.stock.doctype.warehouse.warehouse import apply_warehouse_filter
SLE_COUNT_LIMIT = 10_000
def _estimate_table_row_count(doctype: str):
table = get_table_name(doctype)
return cint(
frappe.db.sql(
f"""select table_rows
from information_schema.tables
where table_name = '{table}' ;"""
)[0][0]
)
SLE_COUNT_LIMIT = 100_000
def execute(filters=None):
if not filters:
filters = {}
sle_count = _estimate_table_row_count("Stock Ledger Entry")
sle_count = frappe.db.estimate_count("Stock Ledger Entry")
if (
sle_count > SLE_COUNT_LIMIT