Skip to content

fix: adjust chunk size of git diff and fix id of item#321

Merged
cx-rogerio-dalot merged 12 commits intomasterfrom
AST-0000-spike-mem-2
Aug 19, 2025
Merged

fix: adjust chunk size of git diff and fix id of item#321
cx-rogerio-dalot merged 12 commits intomasterfrom
AST-0000-spike-mem-2

Conversation

@cx-rogerio-dalot
Copy link
Copy Markdown
Contributor

@cx-rogerio-dalot cx-rogerio-dalot commented Aug 18, 2025

Closes #

Proposed Changes

  • Adjust the size of chunk to at most have 256KB since it improves the regexp handling performance (~15% faster)
  • Fix the id of the removed git diff

Checklist

  • I covered my changes with tests.
  • I Updated the documentation that is affected by my changes:
    • Change in the CLI arguments
    • Change in the configuration file

I submit this contribution under the Apache-2.0 license.

@cx-rogerio-dalot cx-rogerio-dalot requested a review from a team as a code owner August 18, 2025 17:11
@github-actions
Copy link
Copy Markdown

kics-logo

KICS version: v1.7.13

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 13
Files parsed placeholder 13
Files failed to scan placeholder 0
Total executed queries placeholder 53
Queries failed to execute placeholder 0
Execution time placeholder 1

@github-actions
Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Details4b15c8f1-3524-479f-93bd-75f849bb080c

Great job! No new security vulnerabilities introduced in this pull request

@cx-rogerio-dalot cx-rogerio-dalot merged commit f824ca8 into master Aug 19, 2025
14 checks passed
@cx-rogerio-dalot cx-rogerio-dalot deleted the AST-0000-spike-mem-2 branch August 19, 2025 08:46
cx-rogerio-dalot added a commit that referenced this pull request Sep 11, 2025
<!--
Thanks for contributing to 2ms by offering a pull request.
-->

Closes #

**Proposed Changes**

- Adjust the size of chunk to at most have 256KB since it improves the
regexp handling performance (~15% faster)
- Fix the id of the removed git diff

<!--
Please describe the big picture of your changes here. If it fixes a bug
or resolves a feature request, be sure to link to that issue.
-->

**Checklist**

- [ ] I covered my changes with tests.
- [ ] I Updated the documentation that is affected by my changes:
  - [ ] Change in the CLI arguments
  - [ ] Change in the configuration file

I submit this contribution under the Apache-2.0 license.

---------

Co-authored-by: Rogério Dalot <206845486+cx-rogerio-dalot-x@users.noreply.github.com>
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.

2 participants