Skip to content

Improve message.new event parsing memory footprint #6344

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

Open
VelikovPetar wants to merge 16 commits into
base: v6
Choose a base branch
from

Merge branch 'v6' into feature/new_event_parsing

a8c53db
Select commit
Loading
Failed to load commit list.
Open

Improve message.new event parsing memory footprint #6344

Merge branch 'v6' into feature/new_event_parsing
a8c53db
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 6, 2026 in 2m 6s

Quality Gate passed

Issues
3 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 106 in stream-chat-android-client/src/main/java/io/getstream/chat/android/client/parser2/direct/MessageAdapter.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the number of when branches from 41 to at most 30. 

See more on https://sonarcloud.io/project/issues?id=GetStream_stream-chat-android&issues=AZ1uhkkn_fTwIxHcwfUt&open=AZ1uhkkn_fTwIxHcwfUt&pullRequest=6344

Check warning on line 186 in stream-chat-android-state/src/main/java/io/getstream/chat/android/state/plugin/factory/StreamStatePluginFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This function has 11 parameters, which is greater than the 7 authorized. 

See more on https://sonarcloud.io/project/issues?id=GetStream_stream-chat-android&issues=AZ3foIoE-Y0uU4J1P_FY&open=AZ3foIoE-Y0uU4J1P_FY&pullRequest=6344

Check failure on line 37 in stream-chat-android-client/src/main/java/io/getstream/chat/android/client/parser2/direct/NewMessageEventAdapter.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=GetStream_stream-chat-android&issues=AZ1uhkff_fTwIxHcwfUs&open=AZ1uhkff_fTwIxHcwfUs&pullRequest=6344
View more details on SonarQubeCloud