Closed buraksezer closed 1 week ago
:broken_heart: The detected issue is not in one of the allowed statuses :broken_heart:
Detected Status | Open | :x: |
---|---|---|
Allowed Statuses | In Dev,In Code Review,Ready for Testing,In Test,In Progress,In Review | :heavy_check_mark: |
Please ensure your jira story is in one of the allowed statuses
Here are some key observations to aid the review process:
⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪ |
🧪 No relevant tests |
🔒 No security concerns identified |
⚡ Recommended focus areas for review Import Specificity The PR replaces a broad import with specific component imports. Ensure that no other components from the previously imported package are required elsewhere in the application to avoid runtime issues. |
No code suggestions found for the PR.
API Changes
no api changes detected
Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
/release to release-5.7
Working on it! Note that it can take a few minutes.
/release to release-5.7.0
@buraksezer Seems like there is conflict and it require manual merge.
Working on it! Note that it can take a few minutes.
@buraksezer Seems like there is conflict and it require manual merge.
User description
Attempt to fix TT-13607
We should not need to import all components. I also ran
go mod tidy
to reduce the dependency graph.Initial memory usage::
I tested the changes locally with Kafka & labs-streams sample. It works as usual.
PR Type
enhancement, dependencies
Description
stream.go
to include only necessary components (io
andkafka
), reducing memory usage on cold start.go.sum
file by removing unused dependencies, which helps in reducing the dependency graph.Changes walkthrough 📝
stream.go
Optimize component imports to reduce memory usage
ee/middleware/streams/stream.go
components/io
andcomponents/kafka
.go.sum
Clean up and update Go module dependencies
go.sum