Skip to content

Set some runtime fields in FIConetxt to null when the FI is done#9240

Merged
JackieTien97 merged 1 commit intomasterfrom
ReduceContextSize
Mar 8, 2023
Merged

Set some runtime fields in FIConetxt to null when the FI is done#9240
JackieTien97 merged 1 commit intomasterfrom
ReduceContextSize

Conversation

@JackieTien97
Copy link
Contributor

@JackieTien97 JackieTien97 commented Mar 8, 2023

We will keep FIContext in memory for 5 mins, so we only keep necessary infos in FIContext for memory cost.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 8, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@wangchao316 wangchao316 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

3 participants