Skip to content

fix:[NEXT-1893] Skip the AssetType if the ES throws exception#2381

Merged
arunpaladin merged 1 commit into
masterfrom
fix/shipper-es-fix
Oct 31, 2025
Merged

fix:[NEXT-1893] Skip the AssetType if the ES throws exception#2381
arunpaladin merged 1 commit into
masterfrom
fix/shipper-es-fix

Conversation

@arunpaladin
Copy link
Copy Markdown
Collaborator

@arunpaladin arunpaladin commented Oct 31, 2025

fix:NEXT-1893 Skip the AssetType if the ES throws exception

Summary by CodeRabbit

  • Bug Fixes

    • Enhanced error handling and reporting in data processing operations to provide more descriptive error messages and improve system reliability.
  • Tests

    • Updated test infrastructure to properly handle exceptions in data upload validation.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 31, 2025

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

📥 Commits

Reviewing files that changed from the base of the PR and between 591ee0b and 661765d.

📒 Files selected for processing (2)
  • jobs/pacman-data-shipper/src/main/java/com/tmobile/cso/pacman/datashipper/es/ESManager.java (3 hunks)
  • jobs/pacman-data-shipper/src/test/java/com/tmobile/cso/pacman/datashipper/entity/EntityManagerTest.java (1 hunks)
 _____________________________________________
< HD, 4K, 8K...I see bugs in all resolutions. >
 ---------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/shipper-es-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@arunpaladin arunpaladin merged commit 5f71490 into master Oct 31, 2025
59 of 60 checks passed
@arunpaladin arunpaladin deleted the fix/shipper-es-fix branch October 31, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants