wu-sheng commented on PR #10884: URL: https://github.com/apache/skywalking/pull/10884#issuecomment-1581787416
> this PR adds a URI pattern recognition API and limit the implementations to be AI-powered (at least all the names AIPipeline....), but it can actually be any implementation that are nothing related t o AI, e.g. I can even just implement the APIs and perform a naive regular expression match and returns the pattern. First, let's begin with this. AI-powered doesn't mean AI could do things without context. We are not limiting everything, but AFAIK, certain contexts provide better output from every algorithm/model we have known. The idea had been put in [here](https://github.com/apache/skywalking/discussions/10872), especially **Ideas** part. ____ Then back to your summary > the new module can be moved/renamed into a feature-related module, such as uri-grouping-plugin, It is switchable, and it doesn't have to run as the service is only up and running when upstream(server addr) has been set. I just don't want the kernel logic so context. Read **EndpointNameGrouping#L131**, `if (!httpUriRecognitionSvr.isInitialized())`, nothing happens. > we don't have to address the technology (AI) we want to use, we can just provide the API and document that we have an implementation powered by AI. (reminds me the difference between HAS A vs. IS A) We don't as I mentioned. And it is definitely pluggable to use different servers to bind with a simple LLM pattern recognition or OpenAPI commercial API. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
