Skip to content

Display Record Size Information#671

Merged
bunty95 merged 40 commits intoCOVESA:masterfrom
Pavithra-aa-Anand:Feature-DisplayRecordSizeInfo
Feb 18, 2026
Merged

Display Record Size Information#671
bunty95 merged 40 commits intoCOVESA:masterfrom
Pavithra-aa-Anand:Feature-DisplayRecordSizeInfo

Conversation

@Pavithra-aa-Anand
Copy link
Collaborator

Payload, Message and Storage size of a DLT log file is displayed in a dialog while clicking "DLT File size" in file menu

Signed-off by : Pavithra Anand Pavithra.AA.Anand@bti.bmwgroup.com

@alexmucde
Copy link
Collaborator

Creating statistics of DLt Files should not be done by reading again the DLT file in Mainwindow, this must be done directly during the indexing in the QDltFile to prevent a performance issue.
Performance issue must be tested with and without change.

@bunty95
Copy link
Collaborator

bunty95 commented Feb 5, 2026

Should this be merged into main branch?

@Pavithra-aa-Anand
Copy link
Collaborator Author

Should this be merged into main branch?
Yes, it has to be merged.

@bunty95
Copy link
Collaborator

bunty95 commented Feb 6, 2026

Can you provide statistics showing that there is no performance impact for this change. Once provided, I will merge to main

@Pavithra-aa-Anand
Copy link
Collaborator Author

Can you provide statistics showing that there is no performance impact for this change. Once provided, I will merge to main

Tested in Linux with file size of 3GB

Performance statistics:
stat

@bunty95
Copy link
Collaborator

bunty95 commented Feb 18, 2026

Based on provided stats, I will merge with main branch

@bunty95 bunty95 merged commit 5ef1494 into COVESA:master Feb 18, 2026
7 checks passed
@Pavithra-aa-Anand Pavithra-aa-Anand deleted the Feature-DisplayRecordSizeInfo branch February 19, 2026 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments