-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Open
Description
Description
Several TODO comments in the graph-feature-service module indicate incomplete or planned functionality that should be addressed:
TODOs Found:
-
ServerController.scala
// TODO: Disable updateCache after HTL switch to use PresetIntersection endpoint// TODO: Refactor after HTL switch to PresetIntersection
-
ServerGetIntersectionHandler.scala
// TODO: Track all the stats based on PresetFeatureType and update the dashboard
-
ServerWarmupHandler.scala
// TODO: Add the testing accounts to warm-up the service
-
IntersectionValueCalculator.scala
// TODO: for now it only computes intersection size. Will add more feature types
Impact
These TODOs indicate:
- Technical debt that needs to be addressed
- Missing functionality (stats tracking, additional feature types)
- Potential cleanup after migrations
Suggested Actions
- Review if HTL switch to PresetIntersection is complete
- Implement stats tracking for PresetFeatureType
- Add testing accounts for service warmup
- Extend IntersectionValueCalculator with additional feature types
Files Affected
graph-feature-service/src/main/scala/com/twitter/graph_feature_service/server/controllers/ServerController.scalagraph-feature-service/src/main/scala/com/twitter/graph_feature_service/server/handlers/ServerGetIntersectionHandler.scalagraph-feature-service/src/main/scala/com/twitter/graph_feature_service/server/handlers/ServerWarmupHandler.scalagraph-feature-service/src/main/scala/com/twitter/graph_feature_service/util/IntersectionValueCalculator.scala
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels