Subversion Repositories SmartDukaan

Last modification

• Rev 37566 – 28 d 11 h
• Author: amit

• Log message:

  Delist logic v3: require a month of no movement; add CatalogDelistService
  
  Zero stock alone was too weak a test. It fired on SKUs partners were still
  transacting, where the recent movement was often the very sale that emptied the
  stock. v3 additionally requires no movement for a month on either side:
  - SmartDukaan via warehouse.scanNew (3.4M rows, live). Note warehouse.scan is a
  dead Saholic relic - 2,476 rows, last write 2012.
  - any partner via fofo.scan_record, any scan type.
  Phrased as NOT EXISTS so they short-circuit on the first recent row and use the
  (inventoryItemId, scannedAt) / (inventory_item_id, create_timestamp) indexes.
  An item that has never moved still qualifies, which is correct.
  
  Also in v3:
  - the internal pseudo-brands (Dummy, FOC, FOC HANDSET, Live Demo) are no longer
  excluded and are swept like anything else.
  - vendor-PO guard narrowed to status = 1. PARTIALLY_FULFILLED is UNREACHABLE,
  not merely rare: none of the 8 PO-status writes in the codebase sets it, no
  native SQL writes the column, and status = 2 has 0 rows across 51,593+ POs
  back to 2011. Partial receipt is modelled per line (unfulfilledQuantity), so a
  part-received PO sits in READY and is already caught. Documented so it is not
  'restored' later as a supposed gap.
  
  CatalogDelistService/Impl mirrors the migration statement for statement, for the
  daily scheduled run. Bulk SQL rather than per-row updateActiveById +
  publishStatusChange, which would commit to Solr once per catalog (~1,750 times
  on the 2026-09-10 backlog); the job is instead scheduled ahead of the existing
  06:00 full reindex. The dark-model step is deliberately house-wide rather than
  run-scoped so it converges instead of leaving pre-existing dark models on a
  stale RUNNING forever.
  
  rollback_delist_recent_movement_20260910.sql restores the 24 listings / 21
  models that the v2 run delisted before NEW 3 existed; their partners had
  transacted them within the month. Id list frozen, not re-derived.