Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

fix: reuses shared object mapper instead of recreating them #796

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

lburgazzoli
Copy link
Contributor

@lburgazzoli lburgazzoli commented Jul 20, 2023

Fixes #786

@lburgazzoli lburgazzoli requested a review from a team July 20, 2023 14:10
@sonarcloud
Copy link

sonarcloud bot commented Jul 20, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
2.1% 2.1% Duplication

@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Merging #796 (25b06d0) into main (af4fbbd) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #796      +/-   ##
============================================
- Coverage     79.17%   79.16%   -0.01%     
  Complexity      167      167              
============================================
  Files            56       57       +1     
  Lines          3040     3029      -11     
  Branches        529      529              
============================================
- Hits           2407     2398       -9     
+ Misses          417      415       -2     
  Partials        216      216              
Impacted Files Coverage Δ
...api/rest/resource/v1/OpenApiToRestDslResource.java 100.00% <100.00%> (ø)
...o/backend/api/resource/v1/DeploymentsResource.java 84.00% <100.00%> (+0.32%) ⬆️
.../main/java/io/kaoto/backend/camel/KamelHelper.java 100.00% <100.00%> (ø)
...in/java/io/kaoto/backend/camel/KamelPopulator.java 88.63% <100.00%> (-0.09%) ⬇️
...data/parser/step/kamelet/KameletFileProcessor.java 83.13% <100.00%> (ø)
...amel/service/deployment/generator/Representer.java 100.00% <100.00%> (ø)
...elroute/IntegrationDeploymentGeneratorService.java 70.96% <100.00%> (+2.71%) ⬆️
...t/generator/camelroute/IntegrationRepresenter.java 100.00% <100.00%> (ø)
...elet/KameletBindingDeploymentGeneratorService.java 72.89% <100.00%> (-0.74%) ⬇️
...tor/kamelet/KameletDeploymentGeneratorService.java 66.66% <100.00%> (-0.65%) ⬇️
... and 6 more

@lburgazzoli
Copy link
Contributor Author

code coverage is expected to fail at this stage

@Delawen Delawen merged commit 2a8fa44 into KaotoIO:main Jul 21, 2023
@lburgazzoli lburgazzoli deleted the object-mapper branch July 21, 2023 07:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

De-Serialization
2 participants