fix: update migration chart_ds_constraint to have seperate batch op

[hughhhh]

SUMMARY

Getting errors for constraint in same batch operation, so refactoring to commit before adding a new constraint

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #23942 (149c76c) into master (3dc4de4) will increase coverage by 0.13%.
The diff coverage is n/a.

❗ Current head 149c76c differs from pull request most recent head 53bb6ed. Consider uploading reports for the commit 53bb6ed to get more accurate results

@@            Coverage Diff             @@
##           master   #23942      +/-   ##
==========================================
+ Coverage   67.98%   68.12%   +0.13%     
==========================================
  Files        1941     1941              
  Lines       75073    75073              
  Branches     8155     8155              
==========================================
+ Hits        51041    51141     +100     
+ Misses      21949    21849     -100     
  Partials     2083     2083              

Flag	Coverage Δ
hive	53.00% <ø> (?)
mysql	78.82% <ø> (?)
postgres	78.89% <ø> (ø)
presto	52.92% <ø> (ø)
python	82.68% <ø> (+0.27%)	⬆️
sqlite	77.41% <ø> (?)
unit	52.81% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sadpandajoe]

🏷️ preset:2023.17