jbertram commented on PR #4730: URL: https://github.com/apache/activemq-artemis/pull/4730#issuecomment-1878075475
Thinking about this more...I'm more against it than before. I believe that opening this door is going to be bad for usability - both for users and developers. Right now, `*` means **a single word**. If we start accepting "partial words" then what does `*` become? Is it any single _character_? Is it 0 or more of any character (e.g. as it might be in a regular expression)? Also, how to partial words compare to each other in a hierarchy where matches are ordered from general to specific? Would `ab*` be more specific than `a*` when matching `abc`? What about `a*c`? Is that even supported? Where do we draw the line? Furthermore, what do we do with `#`? Should we support partial matches with it? If so, what does that mean? If not, why not? The potential configurations start to expand very quickly and will no doubt add complication to the code, the test-suite, and the documentation. The currently documented functionality is simple & powerful and we should keep it that way. If there's a bug here it's that undocumented behavior is allowed and somewhat functional leading folks to assume it's intentional. I'm not saying we should fix that necessarily, but we should at least consider it so we don't keep letting folks get confused. -- 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]
