Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[COST-4852] feat: adding an exception to handle TrinoErrors. #5177

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bacciotti
Copy link
Contributor

@bacciotti bacciotti commented Jun 19, 2024

Jira Ticket

COST-4852

Description

This change will handle the NoSuchKey error on Trino.

Testing

  1. Checkout Branch
  2. Restart Koku
  3. Try to create and load sample data.

Release Notes

  • proposed release note
* [COST-4852](https://issues.redhat.com/browse/COST-4852) Handling some erros during Trino SQL execution.

Copy link

codecov bot commented Jun 19, 2024

Codecov Report

Attention: Patch coverage is 12.50000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 94.1%. Comparing base (1917175) to head (50019e3).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #5177     +/-   ##
=======================================
- Coverage   94.2%   94.1%   -0.0%     
=======================================
  Files        376     376             
  Lines      31234   31242      +8     
  Branches    3727    3729      +2     
=======================================
- Hits       29410   29407      -3     
- Misses      1161    1171     +10     
- Partials     663     664      +1     

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant