Skip to content

Commit

Permalink
Fix typo and grammar error (apache#2483)
Browse files Browse the repository at this point in the history
  • Loading branch information
kezhenxu94 authored and wu-sheng committed Apr 14, 2019
1 parent 2f8e19f commit c27f36a
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
*/
public abstract class AbstractMessageConsumeInterceptor implements InstanceMethodsAroundInterceptor {

public static final String COMSUMER_OPERATION_NAME_PREFIX = "RocketMQ/";
public static final String CONSUMER_OPERATION_NAME_PREFIX = "RocketMQ/";

@Override
public final void beforeMethod(EnhancedInstance objInst, Method method, Object[] allArguments,
Expand All @@ -50,7 +50,7 @@ public final void beforeMethod(EnhancedInstance objInst, Method method, Object[]
List<MessageExt> msgs = (List<MessageExt>)allArguments[0];

ContextCarrier contextCarrier = getContextCarrierFromMessage(msgs.get(0));
AbstractSpan span = ContextManager.createEntrySpan(COMSUMER_OPERATION_NAME_PREFIX + msgs.get(0).getTopic() + "/Consumer", contextCarrier);
AbstractSpan span = ContextManager.createEntrySpan(CONSUMER_OPERATION_NAME_PREFIX + msgs.get(0).getTopic() + "/Consumer", contextCarrier);

span.setComponent(ComponentsDefine.ROCKET_MQ_CONSUMER);
SpanLayer.asMQ(span);
Expand Down
6 changes: 3 additions & 3 deletions docs/en/guides/Java-Plugin-Development-Guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -221,13 +221,13 @@ ClassMatch represents how to match the target classes, there are 4 ways:
* byName, through the full class name(package name + `.` + class name)
* byClassAnnotationMatch, through the class existed certain annotations.
* byMethodAnnotationMatch, through the class's method existed certain annotations.
* byHierarchyMatch, throught the class's parent classes or interfaces
* byHierarchyMatch, through the class's parent classes or interfaces

**Attentions**:
* Forbid to use `*.class.getName()` to get the class String name. Recommend you to use literal String. This is for
avoiding ClassLoader issues.
* `by*AnnotationMatch` doesn't support the inherited annotations.
* Don't recommend use `byHierarchyMatch`, unless it is really necessary. Because using it may trigger intercepting
* Don't recommend to use `byHierarchyMatch`, unless it is really necessary. Because using it may trigger intercepting
many unexcepted methods, which causes performance issues and concerns.

Example:
Expand Down Expand Up @@ -322,5 +322,5 @@ Please follow there steps:
1. Develop and test.
1. Send the pull request and ask for review.
1. Provide the automatic test cases.
1. The plugin committers approves your plugins after automatic test cases provided and the tests passed in our CI.
1. The plugin committers approve your plugins after automatic test cases provided and the tests passed in our CI.
1. The plugin accepted by SkyWalking.
2 changes: 1 addition & 1 deletion docs/en/guides/storage-extention.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ or `org.apache.skywalking.oap.server.storage.plugin.jdbc.mysql.MySQLStorageProvi

## Redistribution with new storage implementation.
You don't have to clone the main repo just for implementing the storage. You could just easy depend our Apache releases.
Take a look at [SkyAPM/SkyWalking-With-Es5x-Storage](https://github.com/SkyAPM/SkyWalking-With-Es5x-Storage) repo, SkyWalking v6 redistribution with ElasticSearch 5 TCP connection storage implemention.
Take a look at [SkyAPM/SkyWalking-With-Es5x-Storage](https://github.com/SkyAPM/SkyWalking-With-Es5x-Storage) repo, SkyWalking v6 redistribution with ElasticSearch 5 TCP connection storage implementation.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ of [v1(a.k.a. sw3)](Skywalking-Cross-Process-Propagation-Headers-Protocol-v1.md)
The major differences of v2 and v1, comes from SkyWalking's evolution, including
1. Mesh and languages are not same always, some info in headers should be optional.
1. BASE64 encoding required.
1. Sampling flag is including.
1. Sampling flag is included.

## Header Item
* Header Name: `sw6`
Expand Down
4 changes: 2 additions & 2 deletions docs/en/protocols/Trace-Data-Protocol-v2.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,10 @@ For trace format, there are some notices

* EntrySpan
EntrySpan represents a service provider, also the endpoint of server side. As an APM system, we are targeting the
application servers. So almost all the services and MQ-comsumer are EntrySpan(s).
application servers. So almost all the services and MQ-consumer are EntrySpan(s).

* LocalSpan
LocalSpan represents a normal Java method, which don't relate with remote service, neither a MQ producer/comsumer
LocalSpan represents a normal Java method, which don't relate with remote service, neither a MQ producer/consumer
nor a service(e.g. HTTP service) provider/consumer.

* ExitSpan
Expand Down

0 comments on commit c27f36a

Please sign in to comment.