lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

fix(site-finder): db.rollback() в velocity catches + SAVEPOINT zoning/success_rec

Browse source 
velocity.py 3 internal try/except возвращали None БЕЗ db.rollback(). PR #154
SAVEPOINT не помог — exception не propagates из velocity. Postgres tx
aborted внутри, cascade InFailedSqlTransaction на следующих queries.

Fix:
- velocity.py: db.rollback() в 3 catch блоках
- parcels.py: SAVEPOINT обёртки для zoning + success_recommendation

Refs: user reports 2026-05-15 cascade 500
This commit is contained in:
lekss361 2026-05-15 08:40:48 +03:00
parent b13c4be0ed
commit c3cafefb1a
2 changed files with 76 additions and 57 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

116
backend/app/api/v1/parcels.py
View file

@ -1658,33 +1658,34 @@ def analyze_parcel(
"lon": centroid_lon, "lon": centroid_lon,
} }
try: try:
zoning_row = ( with db.begin_nested():
db.execute( zoning_row = (
text(""" db.execute(
SELECT zone_code, zone_name, description, rosreestr_id text("""
FROM pzz_zones_ekb SELECT zone_code, zone_name, description, rosreestr_id
WHERE ST_Within( FROM pzz_zones_ekb
ST_Centroid(ST_GeomFromText(:wkt, 4326)), WHERE ST_Within(
geom ST_Centroid(ST_GeomFromText(:wkt, 4326)),
) geom
LIMIT 1 )
"""), LIMIT 1
{"wkt": geom_wkt}, """),
) {"wkt": geom_wkt},
.mappings() )
.first() .mappings()
) .first()
if zoning_row:
zoning.update(
{
"zone_code": zoning_row["zone_code"],
"zone_name": zoning_row["zone_name"],
"description": zoning_row["description"],
"rosreestr_id": zoning_row["rosreestr_id"],
"data_available": True,
"source": "rosreestr-pkk6-cached",
}
) )
if zoning_row:
zoning.update(
{
"zone_code": zoning_row["zone_code"],
"zone_name": zoning_row["zone_name"],
"description": zoning_row["description"],
"rosreestr_id": zoning_row["rosreestr_id"],
"data_available": True,
"source": "rosreestr-pkk6-cached",
}
)
except Exception as e: except Exception as e:
logger.warning("zoning query failed for %s: %s", cad_num, e) logger.warning("zoning query failed for %s: %s", cad_num, e)
@ -1692,9 +1693,10 @@ def analyze_parcel(
success_recommendation: dict[str, Any] | None = None success_recommendation: dict[str, Any] | None = None
if district_row: if district_row:
try: try:
success_rows = ( with db.begin_nested():
db.execute( success_rows = (
text(""" db.execute(
text("""
SELECT bucket, success_score, n_deals, avg_price_per_m2, avg_area_m2, SELECT bucket, success_score, n_deals, avg_price_per_m2, avg_area_m2,
velocity_z, price_z, area_z velocity_z, price_z, area_z
FROM v_bucket_success_score FROM v_bucket_success_score
@ -1702,35 +1704,35 @@ def analyze_parcel(
ORDER BY success_score DESC ORDER BY success_score DESC
LIMIT 5 LIMIT 5
"""), """),
{"dn": district_row["district_name"]}, {"dn": district_row["district_name"]},
)
.mappings()
.all()
) )
.mappings() if success_rows:
.all() success_recommendation = {
) "district": district_row["district_name"],
if success_rows: "ranking": [
success_recommendation = { {
"district": district_row["district_name"], "bucket": r["bucket"],
"ranking": [ "success_score": round(float(r["success_score"]), 2),
{ "n_deals": int(r["n_deals"]),
"bucket": r["bucket"], "avg_price_per_m2": (
"success_score": round(float(r["success_score"]), 2), int(r["avg_price_per_m2"]) if r["avg_price_per_m2"] else None
"n_deals": int(r["n_deals"]), ),
"avg_price_per_m2": ( "avg_area_m2": (
int(r["avg_price_per_m2"]) if r["avg_price_per_m2"] else None round(float(r["avg_area_m2"]), 1) if r["avg_area_m2"] else None
), ),
"avg_area_m2": ( }
round(float(r["avg_area_m2"]), 1) if r["avg_area_m2"] else None for r in success_rows
), ],
} "top_bucket": dict(success_rows[0]) if success_rows else None,
for r in success_rows "note": (
], "Топ комнатность по 'успешности' = z-scores: velocity×0.5 + price×0.3 "
"top_bucket": dict(success_rows[0]) if success_rows else None, "- area×0.2. Min 30 сделок в группе за 24 мес. "
"note": ( "Используй для квартирографии проекта."
"Топ комнатность по 'успешности' = z-scores: velocity×0.5 + price×0.3 " ),
"- area×0.2. Min 30 сделок в группе за 24 мес. " }
"Используй для квартирографии проекта."
),
}
except Exception as e: except Exception as e:
logger.warning("success_recommendation query failed for %s: %s", cad_num, e) logger.warning("success_recommendation query failed for %s: %s", cad_num, e)
success_recommendation = None success_recommendation = None

17
backend/app/services/site_finder/velocity.py
View file

@ -126,6 +126,13 @@ def compute_velocity(
) )
except Exception: except Exception:
logger.exception("velocity: competitor query failed for wkt=%s", parcel_geom_wkt[:80]) logger.exception("velocity: competitor query failed for wkt=%s", parcel_geom_wkt[:80])
# КРИТИЧНО: rollback aborted transaction чтобы caller (analyze_parcel)
# мог продолжить выполнять последующие queries. Иначе caller получает
# cascade InFailedSqlTransaction на следующей SQL execute.
try:
db.rollback()
except Exception:
pass
return None return None
if not comp_rows: if not comp_rows:
@ -186,6 +193,11 @@ def compute_velocity(
) )
except Exception: except Exception:
logger.exception("velocity: sale_graph query failed for obj_ids=%s", obj_ids[:5]) logger.exception("velocity: sale_graph query failed for obj_ids=%s", obj_ids[:5])
# КРИТИЧНО: rollback aborted transaction (см. competitor query комментарий)
try:
db.rollback()
except Exception:
pass
return None return None
if not sales_rows: if not sales_rows:
@ -311,6 +323,11 @@ def _get_ekb_median(db: Session, months_window: int = 6) -> float | None:
) )
except Exception: except Exception:
logger.warning("velocity: ekb_median query failed, using fallback") logger.warning("velocity: ekb_median query failed, using fallback")
# КРИТИЧНО: rollback aborted transaction
try:
db.rollback()
except Exception:
pass
return None return None
if row and row["median"] is not None: if row and row["median"] is not None: