Skip to content

Conversation

@tom-pytel
Copy link
Contributor

  • Changed fast_path_match() to use Python regex instead of working one character at a time.
  • Also pulled out invariant disable_patterns out of loop in plugins.install().

@kezhenxu94 kezhenxu94 added the enhancement New feature or request label Nov 22, 2020
#


def fast_path_match(pattern: str, path: str):
Copy link
Member

@kezhenxu94 kezhenxu94 Nov 22, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was ported from a Java version, I've just run a simple benchmark test, regex is much more better then the original version, thanks

Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kezhenxu94 kezhenxu94 added this to the 0.5.0 milestone Nov 22, 2020
@kezhenxu94 kezhenxu94 merged commit 5e42df2 into apache:master Nov 22, 2020
@tom-pytel
Copy link
Contributor Author

Hey, can't seem to find a way to open an issue on the NodeJS agent so will ask here, having a problem running the test due to the following:

Step 1/5 : FROM nodejs-dependencies
    ✓ test /home/tom/src/revdebugjs/skywalking-nodejs/tests/plugins/http
Service 'server' failed to build: pull access denied for nodejs-dependencies, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Any idea?

@wu-sheng
Copy link
Member

Hi @tom-pytel we share the issue list in the main repo to make people easier, feel free to submit a discussion here, https://github.com/apache/skywalking/issues with project name as the prefix of the issue title. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants