Subversion Repositories SmartDukaan

Last modification

• Rev 36371 2026-04-24 12:10:46
• Author: amit

• Log message:

  HDFC webhook: INSERT IGNORE on hdfc_payment to fix concurrent-duplicate 500 storm
  
  Replaces persist() with a native INSERT IGNORE so concurrent same-UTR webhook
  retries serialize on the unique-index check, and the loser gets a 0-row no-op
  (warning, not exception). Outer @Transactional session stays clean, Spring
  commits normally, HDFC sees 200 on both the winner (Success) and the loser
  (Duplicate) - retry amplification ends.
  
  Flow:
  persist(hdfcPayment) -> insertIgnore(hdfcPayment) + selectByUtrNo(utr)
  - inserted == 1: proceed with wallet / sidbi side-effects using fetched id
  - inserted == 0: respond Duplicate, skip side-effects (owned by winning tx)
  - defensive: log warn if insertIgnore returned 0 but selectByUtrNo finds
  no row (could mean IGNORE swallowed a non-duplicate issue like truncation)
  
  Also drops HdfcProcessingHelper (r36366) which used REQUIRES_NEW for the same
  goal; that approach required a second JDBC connection and is no longer needed
  with SQL-level idempotency.
  
  Does NOT touch the wallet-side lost-update hole (user_wallet read-modify-write
  without FOR UPDATE). That is a separate commit.