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.scala
graph-feature-service/src/main/scala/com/twitter/graph_feature_service/server/handlers/ServerGetIntersectionHandler.scala
graph-feature-service/src/main/scala/com/twitter/graph_feature_service/server/handlers/ServerWarmupHandler.scala
graph-feature-service/src/main/scala/com/twitter/graph_feature_service/util/IntersectionValueCalculator.scala
Description
Several TODO comments in the
graph-feature-servicemodule 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 PresetIntersectionServerGetIntersectionHandler.scala
// TODO: Track all the stats based on PresetFeatureType and update the dashboardServerWarmupHandler.scala
// TODO: Add the testing accounts to warm-up the serviceIntersectionValueCalculator.scala
// TODO: for now it only computes intersection size. Will add more feature typesImpact
These TODOs indicate:
Suggested Actions
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