[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716359892 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716353095 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716354577 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716349472 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716349472 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716349472 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716348556 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking-java] kezhenxu94 commented on pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
kezhenxu94 commented on pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#issuecomment-927516975 > @kezhenxu94 use the guava `ImmutableMap` is a good idea, but failed the checkstyle. ok. We need some adjustments of the check style settings -- This is an

[GitHub] [skywalking-java] VictorZeng commented on pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#issuecomment-927515473 @kezhenxu94 use the guava `ImmutableMap` is a good idea, but failed the checkstyle. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716345457 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716343890 ## File path: test/e2e-v2/script/prepare/setup-oap/download-mysql.sh ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716342690 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716342888 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716342690 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716342607 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716342030 ## File path: .github/actions/infra-e2e-test/action.yml ## @@ -0,0 +1,67 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [skywalking] wu-sheng commented on pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#issuecomment-927505571 One more, please add check for `service dependency`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716341590 ## File path: .github/actions/infra-e2e-test/action.yml ## @@ -0,0 +1,67 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [skywalking-java] VictorZeng commented on pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#issuecomment-927505134 > > Apart from the comments inline, I'd also like to discuss whether we should also include part of the JSON string / bytes in the trace, for example, collect the first

[GitHub] [skywalking-java] wu-sheng commented on pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
wu-sheng commented on pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#issuecomment-927503766 > Apart from the comments inline, I'd also like to discuss whether we should also include part of the JSON string / bytes in the trace, for example, collect the first 64

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716338304 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/jackson/define/AbstractInstrumentation.java

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716335046 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716334921 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/jackson/define/AbstractInstrumentation.java

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716334554 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716334075 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716334040 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716333779 ## File path: test/e2e-v2/cases/satellite/native-protocols/docker-compose.yml ## @@ -0,0 +1,111 @@ +# Licensed to the Apache Software Foundation

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716333704 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,26 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716333120 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/jackson/BasicMethodsInterceptor.java

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716332508 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/pom.xml ## @@ -0,0 +1,58 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716331656 ## File path:

[GitHub] [skywalking-java] VictorZeng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716328918 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/jackson/define/AbstractInstrumentation.java

[GitHub] [skywalking-java] wu-sheng commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716327178 ## File path: apm-sniffer/optional-plugins/jackson-2.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/jackson/define/AbstractInstrumentation.java

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716326393 ## File path: test/e2e-v2/script/docker-compose/base-compose.yml ## @@ -0,0 +1,102 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716325818 ## File path: test/e2e-v2/script/docker-compose/base-compose.yml ## @@ -0,0 +1,102 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking-java] kezhenxu94 commented on a change in pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#discussion_r716321760 ## File path:

[GitHub] [skywalking] mrproliu commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716324000 ## File path: test/e2e-v2/script/docker-compose/base-compose.yml ## @@ -0,0 +1,102 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking-satellite] wu-sheng merged pull request #72: Fix wrong Docker registry

2021-09-26 Thread GitBox
wu-sheng merged pull request #72: URL: https://github.com/apache/skywalking-satellite/pull/72 -- 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:

[skywalking-satellite] branch main updated: Fix wrong Docker registry (#72)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git The following commit(s) were added to refs/heads/main by this push: new 1f3c08a Fix wrong Docker registry

[skywalking-satellite] branch docker-snapshot updated (caefe3c -> 661b85a)

2021-09-26 Thread kezhenxu94
This is an automated email from the ASF dual-hosted git repository. kezhenxu94 pushed a change to branch docker-snapshot in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git. discard caefe3c Fix wrong Docker registry add 661b85a Fix wrong Docker registry This

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716314551 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716314081 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[skywalking-website] branch master updated (d844c07 -> 137b932)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/skywalking-website.git. from d844c07 Add a missing repo add 137b932 Update project page styles (#339) No new revisions were

[GitHub] [skywalking-website] wu-sheng merged pull request #339: Update project page styles

2021-09-26 Thread GitBox
wu-sheng merged pull request #339: URL: https://github.com/apache/skywalking-website/pull/339 -- 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:

[skywalking-satellite] branch docker-snapshot created (now caefe3c)

2021-09-26 Thread kezhenxu94
This is an automated email from the ASF dual-hosted git repository. kezhenxu94 pushed a change to branch docker-snapshot in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git. at caefe3c Fix wrong Docker registry This branch includes the following new commits:

[skywalking-satellite] 01/01: Fix wrong Docker registry

2021-09-26 Thread kezhenxu94
This is an automated email from the ASF dual-hosted git repository. kezhenxu94 pushed a commit to branch docker-snapshot in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git commit caefe3cde4edd596a156d0a805043cd202d34eb5 Author: kezhenxu94 AuthorDate: Mon Sep 27 10:15:29

[GitHub] [skywalking] VictorZeng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
VictorZeng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716312633 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[GitHub] [skywalking-website] Jtrust opened a new pull request #339: Update project page styles

2021-09-26 Thread GitBox
Jtrust opened a new pull request #339: URL: https://github.com/apache/skywalking-website/pull/339 screenshot: ![image](https://user-images.githubusercontent.com/21056967/134835684-cc2b41bc-4c83-4882-af65-5dcaf911054b.png) -- This is an automated message from the Apache Git

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716312089 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716311714 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#discussion_r716311448 ## File path: changes/changes-8.8.0.md ## @@ -75,7 +75,7 @@ Release Notes. low down metrics flush period added in 8.7.0. * Fix `Slow SQL

[GitHub] [skywalking-website] wu-sheng opened a new pull request #338: Set up 8.8.0 release event.

2021-09-26 Thread GitBox
wu-sheng opened a new pull request #338: URL: https://github.com/apache/skywalking-website/pull/338 Pending on release vote, it should be done tomorrow. -- 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

[skywalking-website] branch 8.8.0-release created (now a616a4e)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch 8.8.0-release in repository https://gitbox.apache.org/repos/asf/skywalking-website.git. at a616a4e Set up 8.8.0 release event. No new revisions were added by this update.

[GitHub] [skywalking] VictorZeng commented on pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
VictorZeng commented on pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#issuecomment-927450145 Maybe the plugin should be in version 8.8.0. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [skywalking] wu-sheng commented on pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
wu-sheng commented on pull request #7813: URL: https://github.com/apache/skywalking/pull/7813#issuecomment-927445851 Pending on https://github.com/apache/skywalking-java/pull/39 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [skywalking-satellite] mrproliu merged pull request #71: Fix branch name

2021-09-26 Thread GitBox
mrproliu merged pull request #71: URL: https://github.com/apache/skywalking-satellite/pull/71 -- 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:

[skywalking-satellite] branch main updated: Fix branch name (#71)

2021-09-26 Thread liuhan
This is an automated email from the ASF dual-hosted git repository. liuhan pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git The following commit(s) were added to refs/heads/main by this push: new fbd92ab Fix branch name (#71)

[skywalking-satellite] 01/01: Fix branch name

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch wu-sheng-patch-1 in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git commit f7714779868a24340ef2fa76955efb9c3fbb9904 Author: 吴晟 Wu Sheng AuthorDate: Mon Sep 27 09:43:53

[skywalking-satellite] branch wu-sheng-patch-1 created (now f771477)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch wu-sheng-patch-1 in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git. at f771477 Fix branch name This branch includes the following new commits: new f771477

[GitHub] [skywalking-java] VictorZeng opened a new pull request #39: Add an agent plugin to support Jackson

2021-09-26 Thread GitBox
VictorZeng opened a new pull request #39: URL: https://github.com/apache/skywalking-java/pull/39 ### Add an agent plugin to support `Jackson` - [x] Add a test case for the new plugin, refer to [the

[GitHub] [skywalking-satellite] wu-sheng merged pull request #70: Add publish docker when each PR merged

2021-09-26 Thread GitBox
wu-sheng merged pull request #70: URL: https://github.com/apache/skywalking-satellite/pull/70 -- 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:

[skywalking-satellite] branch main updated: Add publish docker when each PR merged (#70)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/skywalking-satellite.git The following commit(s) were added to refs/heads/main by this push: new a3ce2ec Add publish docker when

[GitHub] [skywalking] VictorZeng opened a new pull request #7813: Add component definition for Jackson.

2021-09-26 Thread GitBox
VictorZeng opened a new pull request #7813: URL: https://github.com/apache/skywalking/pull/7813 - [x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #7812 . - [x] Update the [`CHANGES`

[GitHub] [skywalking-satellite] kezhenxu94 commented on a change in pull request #70: Add publish docker when each PR merged

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #70: URL: https://github.com/apache/skywalking-satellite/pull/70#discussion_r716300771 ## File path: .github/workflows/publish-docker.yaml ## @@ -0,0 +1,51 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [skywalking] VictorZeng opened a new issue #7812: [Feature] Add Jackson plugin

2021-09-26 Thread GitBox
VictorZeng opened a new issue #7812: URL: https://github.com/apache/skywalking/issues/7812 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/skywalking/issues?q=is%3Aissue) and found no similar feature requirement. ### Description

[GitHub] [skywalking] wankai123 commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wankai123 commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716299511 ## File path: test/e2e-v2/script/docker-compose/base-compose.yml ## @@ -0,0 +1,102 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716296604 ## File path: test/e2e-v2/cases/satellite/Dockerfile.satellite ## @@ -0,0 +1,36 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] wu-sheng merged pull request #7811: Fix words in alarm doc.

2021-09-26 Thread GitBox
wu-sheng merged pull request #7811: URL: https://github.com/apache/skywalking/pull/7811 -- 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:

[skywalking] branch master updated: Fix words in alarm doc (#7811)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/skywalking.git The following commit(s) were added to refs/heads/master by this push: new 96829da Fix words in alarm doc (#7811)

[GitHub] [skywalking] wu-sheng opened a new pull request #7811: Fix words in alarm doc.

2021-09-26 Thread GitBox
wu-sheng opened a new pull request #7811: URL: https://github.com/apache/skywalking/pull/7811 - [ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #. - [ ] Update the [`CHANGES`

[skywalking] 01/01: Update backend-alarm.md

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch wu-sheng-patch-1 in repository https://gitbox.apache.org/repos/asf/skywalking.git commit 904a86419ce1556c3da38bff47c93c46082024e0 Author: 吴晟 Wu Sheng AuthorDate: Mon Sep 27 08:18:04 2021 +0800

[skywalking] branch wu-sheng-patch-1 created (now 904a864)

2021-09-26 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch wu-sheng-patch-1 in repository https://gitbox.apache.org/repos/asf/skywalking.git. at 904a864 Update backend-alarm.md This branch includes the following new commits: new 904a864

[GitHub] [skywalking] wu-sheng commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
wu-sheng commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716280380 ## File path: test/e2e-v2/java-test-service/Dockerfile.consumer ## @@ -0,0 +1,26 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [skywalking] sonatype-lift[bot] commented on a change in pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #7810: URL: https://github.com/apache/skywalking/pull/7810#discussion_r716238214 ## File path: test/e2e-v2/java-test-service/e2e-service-provider/pom.xml ## @@ -0,0 +1,127 @@ + + + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] [skywalking] mrproliu opened a new pull request #7810: Support E2E testing satellite native protocols

2021-09-26 Thread GitBox
mrproliu opened a new pull request #7810: URL: https://github.com/apache/skywalking/pull/7810 ### #7368 , Start using `skywalking-infra-e2e` for e2e testing - [ ] If this is non-trivial feature, paste the links/URLs to the design doc. - [ ] Update the documentation to include this new

[GitHub] [skywalking] kezhenxu94 closed issue #7761: [Bug] plugin test script doesn't correctly run `sed` / `gsed` command

2021-09-26 Thread GitBox
kezhenxu94 closed issue #7761: URL: https://github.com/apache/skywalking/issues/7761 -- 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:

[GitHub] [skywalking-kubernetes-event-exporter] kezhenxu94 commented on a change in pull request #11: support tls for SkyWalking client

2021-09-26 Thread GitBox
kezhenxu94 commented on a change in pull request #11: URL: https://github.com/apache/skywalking-kubernetes-event-exporter/pull/11#discussion_r716207138 ## File path: pkg/exporter/skywalking.go ## @@ -67,7 +78,39 @@ func (exporter *SkyWalking) Init(ctx context.Context) error {

[skywalking] branch master updated: Remove unused defines (#7807)

2021-09-26 Thread kezhenxu94
This is an automated email from the ASF dual-hosted git repository. kezhenxu94 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/skywalking.git The following commit(s) were added to refs/heads/master by this push: new c53ee80 Remove unused defines (#7807)

[GitHub] [skywalking] kezhenxu94 merged pull request #7807: remove unused defines

2021-09-26 Thread GitBox
kezhenxu94 merged pull request #7807: URL: https://github.com/apache/skywalking/pull/7807 -- 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:

[GitHub] [skywalking-client-js] Fine0830 merged pull request #74: fix: update firstReportedError for SPA

2021-09-26 Thread GitBox
Fine0830 merged pull request #74: URL: https://github.com/apache/skywalking-client-js/pull/74 -- 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:

[skywalking-client-js] branch master updated: fix: update firstReportedError for SPA (#74)

2021-09-26 Thread qiuxiafan
This is an automated email from the ASF dual-hosted git repository. qiuxiafan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/skywalking-client-js.git The following commit(s) were added to refs/heads/master by this push: new bc36f52 fix: update

[GitHub] [skywalking-client-js] Fine0830 opened a new pull request #74: fix: update firstReportedError for SPA

2021-09-26 Thread GitBox
Fine0830 opened a new pull request #74: URL: https://github.com/apache/skywalking-client-js/pull/74 This PR fixes `firstReportedError` don't be marked `true`, when the route has changed in SPA scenario. Screenshot

[GitHub] [skywalking] wu-sheng closed issue #7808: [Bug] The traceId get from logstash are all mixed up accross multiple tomcat threads

2021-09-26 Thread GitBox
wu-sheng closed issue #7808: URL: https://github.com/apache/skywalking/issues/7808 -- 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:

[GitHub] [skywalking] whohasthis opened a new issue #7808: [Bug] The traceId get from logstash are all mixed up accross multiple tomcat threads

2021-09-26 Thread GitBox
whohasthis opened a new issue #7808: URL: https://github.com/apache/skywalking/issues/7808 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/skywalking/issues?q=is%3Aissue) and found no similar issues. ### Apache SkyWalking Component

[GitHub] [skywalking-banyandb] hanahmily merged pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
hanahmily merged pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49 -- 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:

[skywalking-banyandb] branch main updated: Implement stream query v2 (#49)

2021-09-26 Thread hanahmily
This is an automated email from the ASF dual-hosted git repository. hanahmily pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/skywalking-banyandb.git The following commit(s) were added to refs/heads/main by this push: new fb25f23 Implement stream query v2

[GitHub] [skywalking-banyandb] lujiajing1126 commented on a change in pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
lujiajing1126 commented on a change in pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49#discussion_r716198949 ## File path: banyand/query/v2/query.go ## @@ -0,0 +1,41 @@ +// Licensed to Apache Software Foundation (ASF) under one or more contributor

[GitHub] [skywalking] wu-sheng commented on pull request #7807: remove unused defines

2021-09-26 Thread GitBox
wu-sheng commented on pull request #7807: URL: https://github.com/apache/skywalking/pull/7807#issuecomment-927297871 Good catch. -- 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

[GitHub] [skywalking] JaredTan95 opened a new pull request #7807: remove unused defines

2021-09-26 Thread GitBox
JaredTan95 opened a new pull request #7807: URL: https://github.com/apache/skywalking/pull/7807 - [ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #. - [ ] Update the [`CHANGES`

[skywalking] 01/01: remove unused defines

2021-09-26 Thread tanjian
This is an automated email from the ASF dual-hosted git repository. tanjian pushed a commit to branch clean_up in repository https://gitbox.apache.org/repos/asf/skywalking.git commit 55b04589d50ef9f43bf759d488537d1343120bbf Author: JaredTan95 AuthorDate: Sun Sep 26 20:18:48 2021 +0800

[skywalking] branch clean_up created (now 55b0458)

2021-09-26 Thread tanjian
This is an automated email from the ASF dual-hosted git repository. tanjian pushed a change to branch clean_up in repository https://gitbox.apache.org/repos/asf/skywalking.git. at 55b0458 remove unused defines This branch includes the following new commits: new 55b0458 remove

[GitHub] [skywalking-banyandb] lujiajing1126 commented on a change in pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
lujiajing1126 commented on a change in pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49#discussion_r716184523 ## File path: pkg/query/v2/logical/analyzer.go ## @@ -0,0 +1,262 @@ +// Licensed to Apache Software Foundation (ASF) under one or more

[GitHub] [skywalking-banyandb] lujiajing1126 commented on a change in pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
lujiajing1126 commented on a change in pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49#discussion_r716179169 ## File path: pkg/query/v2/logical/common.go ## @@ -19,41 +19,32 @@ package logical import ( "bytes" - "time" - -

[skywalking-website] branch asf-site updated: deploy: d844c07fd23d8e7ef400b29c9db2456cde8f5015

2021-09-26 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/skywalking-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 34772d1 deploy:

[GitHub] [skywalking] wu-sheng closed issue #7802: [Bug] instance_jvm_memory_heap and meter_vm_filesystem_percentage rule not work

2021-09-26 Thread GitBox
wu-sheng closed issue #7802: URL: https://github.com/apache/skywalking/issues/7802 -- 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:

[GitHub] [skywalking] wu-sheng commented on issue #7802: [Bug] instance_jvm_memory_heap and meter_vm_filesystem_percentage rule not work

2021-09-26 Thread GitBox
wu-sheng commented on issue #7802: URL: https://github.com/apache/skywalking/issues/7802#issuecomment-927270289 Could you read the document first? I don't think `period=1` is a good practice. And what does `include-labels:`=`/` mean? Moving to discussion. -- This is an automated

[GitHub] [skywalking] davidricardo1026 opened a new issue #7802: [Bug] instance_jvm_memory_heap and meter_vm_filesystem_percentage rule not work

2021-09-26 Thread GitBox
davidricardo1026 opened a new issue #7802: URL: https://github.com/apache/skywalking/issues/7802 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/skywalking/issues?q=is%3Aissue) and found no similar issues. ### Apache SkyWalking

[GitHub] [skywalking-banyandb] lujiajing1126 commented on a change in pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
lujiajing1126 commented on a change in pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49#discussion_r716173488 ## File path: pkg/query/v2/logical/common.go ## @@ -19,41 +19,32 @@ package logical import ( "bytes" - "time" - -

[GitHub] [skywalking-banyandb] lujiajing1126 commented on a change in pull request #49: Implement stream query v2

2021-09-26 Thread GitBox
lujiajing1126 commented on a change in pull request #49: URL: https://github.com/apache/skywalking-banyandb/pull/49#discussion_r716173418 ## File path: pkg/query/v2/logical/iter.go ## @@ -0,0 +1,104 @@ +// Licensed to Apache Software Foundation (ASF) under one or more

[GitHub] [skywalking-client-js] wu-sheng merged pull request #73: fix: update conditions for traceSDKInternal

2021-09-26 Thread GitBox
wu-sheng merged pull request #73: URL: https://github.com/apache/skywalking-client-js/pull/73 -- 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:

  1   2   >