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.

Sign up for GitHub

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

Jump to bottom

fix: memory issue crash happening on CloudModeProcessor #544

Merged
merged 13 commits into from
Nov 7, 2024
Merged

fix: memory issue crash happening on CloudModeProcessor #544

merged 13 commits into from
Nov 7, 2024
+88 −88

Conversation

SKannaniOS
Copy link
Contributor

Description

  • This PR resolves a significant memory issue that was causing crashes by implementing logic to catch potential errors and providing users with detailed debug logs for better insight into the issue.

@SKannaniOS SKannaniOS added the v1 label Oct 15, 2024
@SKannaniOS SKannaniOS self-assigned this Oct 15, 2024
@SKannaniOS SKannaniOS requested a review from a team as a code owner October 15, 2024 12:49
@SKannaniOS SKannaniOS changed the title fix: memory issue crash happening on CloudModeProcessor fix: memory issue crash happening on CloudModeProcessor Oct 15, 2024
ChryssaAliferi
ChryssaAliferi previously approved these changes Oct 15, 2024
View reviewed changes
@SKannaniOS SKannaniOS changed the title fix: memory issue crash happening on CloudModeProcessor fix: memory issue crash happening on CloudModeProcessor class Oct 15, 2024
@SKannaniOS SKannaniOS changed the title fix: memory issue crash happening on CloudModeProcessor class fix: memory issue crash happening on CloudModeProcessor Oct 15, 2024
saikumarrs
saikumarrs previously approved these changes Oct 15, 2024
View reviewed changes
@SKannaniOS
chore: update version number for beta release
8cfae9c
@SKannaniOS
Merge branch 'beta-release/sdk-2526-resolve-the-memory-issue-causing-…
8e8db57 
…crash' of github.com:rudderlabs/rudder-sdk-ios into beta-release/sdk-2526-resolve-the-memory-issue-causing-crash
@SKannaniOS
Merge branch 'develop' of github.com:rudderlabs/rudder-sdk-ios into b…
11556ad 
…eta-release/sdk-2526-resolve-the-memory-issue-causing-crash
1abhishekpandey
1abhishekpandey previously approved these changes Nov 5, 2024
View reviewed changes
ChryssaAliferi
ChryssaAliferi previously approved these changes Nov 5, 2024
View reviewed changes
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Nov 7, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
60.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@SKannaniOS SKannaniOS merged commit a71ac44 into develop Nov 7, 2024
11 of 12 checks passed
@SKannaniOS SKannaniOS deleted the beta-release/sdk-2526-resolve-the-memory-issue-causing-crash branch November 7, 2024 14:18
@devops-github-rudderstack devops-github-rudderstack mentioned this pull request Nov 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChryssaAliferi ChryssaAliferi ChryssaAliferi approved these changes

@saikumarrs saikumarrs saikumarrs left review comments

@1abhishekpandey 1abhishekpandey 1abhishekpandey left review comments

Assignees

@SKannaniOS SKannaniOS

Labels
v1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@SKannaniOS @ChryssaAliferi @1abhishekpandey @saikumarrs