Skip to content

CRJVM 205 : Forcer l'utilisation du FetchType LAZY sur les collections dans les Entity JPA - #125

Closed
misterdz-mh wants to merge 9 commits into
green-code-initiative:mainfrom
Qelai:CRJVM205-java
Closed

CRJVM 205 : Forcer l'utilisation du FetchType LAZY sur les collections dans les Entity JPA#125
misterdz-mh wants to merge 9 commits into
green-code-initiative:mainfrom
Qelai:CRJVM205-java

Conversation

@misterdz-mh

Copy link
Copy Markdown

Implémentation de la règle CRJVM 205

@sonarqubecloud

sonarqubecloud Bot commented Apr 6, 2023

Copy link
Copy Markdown

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

82.2% 82.2% Coverage
0.0% 0.0% Duplication

@wokier

wokier commented Apr 6, 2023

Copy link
Copy Markdown

Duplicates #155

@MP-Aubay MP-Aubay added 🗃️ rule rule improvment or rule development or bug java 🏆 challenge2023 🏆 Work done during the ecoCode Challenge 2023 labels Apr 7, 2023
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has no activity for 30 days.
Please add a comment if you want to keep the issue open. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Jun 26, 2023
@dedece35 dedece35 added the duplicate This issue or pull request already exists label Sep 22, 2023
@github-actions github-actions Bot removed the stale label Sep 23, 2023
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has no activity for 30 days.
Please add a comment if you want to keep the issue open. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Oct 24, 2023
@dedece35
dedece35 marked this pull request as draft March 9, 2025 15:27
@dedece35

dedece35 commented Mar 9, 2025

Copy link
Copy Markdown
Member

I marked this PR as draft because of duplication (see above)

@github-actions github-actions Bot removed the stale label Mar 10, 2025
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has no activity for 60 days.
Please add a comment if you want to keep the issue open. Thank you for your contributions!

@dedece35

dedece35 commented Aug 7, 2026

Copy link
Copy Markdown
Member

Hi @misterdz-mh, @wokier

several PR have worked on this same subject (FetchType Lazy instead of Eager for OneToMany et ManyToMany relations).

After worked on a complete analysis to check if this rule is relevant :

  • first, I found that the rule already exists in SonarQube as a built-in rule : S6904
  • secondly, no proof was found to check relevancy
  • thirdly, I decided to launch an analysis with EnergyTracer tool which can do real measures

Thus I created a new PR to :

  • add this analysis
  • clean all old references to this same subject

Here is the PR : #487

Sorry, but I have to close this PR.
thank you for the work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗃️ rule rule improvment or rule development or bug 🏆 challenge2023 🏆 Work done during the ecoCode Challenge 2023 duplicate This issue or pull request already exists java

Projects

Development

Successfully merging this pull request may close these issues.

5 participants