[GitHub] [apisix] spacewander commented on a change in pull request #3987: docs: add more example about how to use the router radixtree-uri

[GitBox <gi...@apache.org>]

spacewander commented on a change in pull request #3987:
URL: https://github.com/apache/apisix/pull/3987#discussion_r607570179



##########
File path: docs/en/latest/router-radixtree.md
##########
@@ -71,7 +71,96 @@ Here are the rules:
 |/blog/foo/gloo | `/blog/foo/*` |
 |/blog/bar | not match |
 
-#### 4. Parameter match
+#### 4. Different routes have the same uri

Review comment:
       We should quote the `uri`, as we mean a field of Route instead of URI.

##########
File path: docs/en/latest/router-radixtree.md
##########
@@ -71,7 +71,96 @@ Here are the rules:
 |/blog/foo/gloo | `/blog/foo/*` |
 |/blog/bar | not match |
 
-#### 4. Parameter match
+#### 4. Different routes have the same uri
+
+When different routes have the same uri, you can set the priority attribute of the route to determine which route to match first, or add matching rules to distinguish different routes.
+
+1. Different routes have the same uri and set the `priority` attribute
+
+Create two routes with different `priority` values ​​(the larger the value, the higher the priority).

Review comment:
       We should mention the `priority` field is prior to the "other matching rules"

##########
File path: docs/en/latest/router-radixtree.md
##########
@@ -71,7 +71,96 @@ Here are the rules:
 |/blog/foo/gloo | `/blog/foo/*` |
 |/blog/bar | not match |
 
-#### 4. Parameter match
+#### 4. Different routes have the same uri
+
+When different routes have the same uri, you can set the priority attribute of the route to determine which route to match first, or add matching rules to distinguish different routes.

Review comment:
       "add other matching rules"




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org