-
Notifications
You must be signed in to change notification settings - Fork 6
Description
Is your feature request related to a problem? Please describe
Potential features which are suggested by comments in code
*** Thread safety in GraphManager.buildGraphByClassifier(GraphClassifier classifier)
// TODO: thread safety
Seems like there were concerns about thread safety in that class, investigation is required
*** Infinity loop protection in RepeatStep.loopHasNext()
// TODO infinity loop
So far no signs has been seen that we need that protection, but worth thinking about it
*** Align PathStep.map() with best practices
// TODO mutable structure. No way
Aligning this part to best practices might bring benefits, but doesn't look like it is feasible to work on that part right now
*** Figure out the source of domains with broken mokedFor field in DefaultMockFieldExtractor.getMockedForSet(Graph origin, JsonObject mock)
// TODO: Figure out how do we get mock domains and their purpose. They are getting caught with this error
Might be not an actual issue, requires investigation of latest logs from real graphs.
*** Sort out inconsistency in RecreateHttpDependenciesTask.findComponentVerticesByDnsNames(Graph graph, Set requiredDnsNames)
// TODO: Move the cleanup outside, it shouldn't be the part of searching process
Contradiction with best practices, not critical for fixing, but still good to be refactored.
*** Implement save result in SaveResultCommand
// TODO: Implement text format conversion
Currently this cli command of query doesn't save anything. So that might be implemented accordingly
This doesn't look like a desired feature by users
*** Potential optimizations in DefaultGraphTraversal.apply(Admin traversal)
// ================================
// ================================
// ================================
// ================================
// ================================
// TODO
Doesn't look feasible at this point. Adding implementation there could be helpful only in case if some optimization will be needed at some point.
*** GraphClassifierImpl.getUnprocessedAppVerIds() implementation
// TODO: implement
Need separate investigation for potential uses.
*** Introduction of application location cache AppDBIntegrationImpl
// TODO? application location cache
Necessity of this cache is to be determined. Might be deprecated with deprecation of "AppDB"
Describe the solution you'd like
The list above is for future considerations
Describe alternatives you've considered
No response
Additional information
No response
Metadata
Metadata
Assignees
Labels
Type
Projects
Status