[skywalking] branch master updated: Correlation protocol implement (#4555)

2020-03-25 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 569baf3 Correlation protocol implement

[skywalking] branch remove-endpoint-register updated (5e78bd0 -> ecbdefd)

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git. from 5e78bd0 Make startup successful and persistence works. add e7f66c9 [CI] Reorganize the

[GitHub] [skywalking] wu-sheng opened a new pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory

2020-03-25 Thread GitBox
wu-sheng opened a new pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory URL: https://github.com/apache/skywalking/pull/4570 If you are familiar with the latest OAP core, you will know, we register id for the endpoint. But this design has

[GitHub] [skywalking] Heguoya commented on issue #4571: add my company

2020-03-25 Thread GitBox
Heguoya commented on issue #4571: add my company URL: https://github.com/apache/skywalking/pull/4571#issuecomment-604208286 add my company This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [skywalking] wu-sheng commented on issue #4517: fix elasticsearch-5.x-plugin when use es6.x TransportClient error

2020-03-25 Thread GitBox
wu-sheng commented on issue #4517: fix elasticsearch-5.x-plugin when use es6.x TransportClient error URL: https://github.com/apache/skywalking/pull/4517#issuecomment-604242785 @aderm Please recheck this PR. This is an

[skywalking] branch remove-endpoint-register updated (ecbdefd -> 2baff1a)

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git. from ecbdefd Merge branch 'master' into remove-endpoint-register add 2baff1a Fix dead codes

[GitHub] [skywalking] wu-sheng commented on a change in pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory

2020-03-25 Thread GitBox
wu-sheng commented on a change in pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory URL: https://github.com/apache/skywalking/pull/4570#discussion_r398330528 ## File path:

[GitHub] [skywalking] codecov-io edited a comment on issue #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory

2020-03-25 Thread GitBox
codecov-io edited a comment on issue #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory URL: https://github.com/apache/skywalking/pull/4570#issuecomment-604210849 # [Codecov](https://codecov.io/gh/apache/skywalking/pull/4570?src=pr=h1) Report >

[skywalking] branch master updated: Add yunda express to the powered by page. (#4571)

2020-03-25 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 cb5ff88 Add yunda express to the powered

[GitHub] [skywalking] wu-sheng merged pull request #4571: Add yunda express to the powered by page.

2020-03-25 Thread GitBox
wu-sheng merged pull request #4571: Add yunda express to the powered by page. URL: https://github.com/apache/skywalking/pull/4571 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [skywalking] yogaling opened a new issue #4572: some query and monitoring features

2020-03-25 Thread GitBox
yogaling opened a new issue #4572: some query and monitoring features URL: https://github.com/apache/skywalking/issues/4572 Please answer these questions before submitting your issue. - Why do you submit this issue? - [ ] Question or discussion - [ ] Bug - [ ] Requirement

[GitHub] [skywalking] Heguoya opened a new pull request #4571: add my company

2020-03-25 Thread GitBox
Heguoya opened a new pull request #4571: add my company URL: https://github.com/apache/skywalking/pull/4571 Please answer these questions before submitting pull request - Why submit this pull request? - [ ] Bug fix - [ ] New feature provided - [ ] Improve performance

[GitHub] [skywalking] kezhenxu94 commented on a change in pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory

2020-03-25 Thread GitBox
kezhenxu94 commented on a change in pull request #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory URL: https://github.com/apache/skywalking/pull/4570#discussion_r398319719 ## File path:

[GitHub] [skywalking] codecov-io commented on issue #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory

2020-03-25 Thread GitBox
codecov-io commented on issue #4570: [WIP][Inventory Design Change] Remove endpoint register and endpoint inventory URL: https://github.com/apache/skywalking/pull/4570#issuecomment-604210849 # [Codecov](https://codecov.io/gh/apache/skywalking/pull/4570?src=pr=h1) Report > Merging

[GitHub] [skywalking] kezhenxu94 merged pull request #4555: Correlation protocol implement

2020-03-25 Thread GitBox
kezhenxu94 merged pull request #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[skywalking] branch remove-endpoint-register updated: Make startup successful and persistence works.

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git The following commit(s) were added to refs/heads/remove-endpoint-register by this push: new

[skywalking] branch remove-endpoint-register updated (2baff1a -> f27e9b9)

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a change to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git. from 2baff1a Fix dead codes in the MySQLTableInstaller. add f27e9b9 Fix mischange. No new

[GitHub] [skywalking-nginx-lua] wu-sheng merged pull request #26: Fix typo

2020-03-25 Thread GitBox
wu-sheng merged pull request #26: Fix typo URL: https://github.com/apache/skywalking-nginx-lua/pull/26 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[skywalking-nginx-lua] branch master updated: Fix typo (#26)

2020-03-25 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-nginx-lua.git The following commit(s) were added to refs/heads/master by this push: new 16b51a7 Fix typo (#26) 16b51a7

[GitHub] [skywalking-nginx-lua] WALL-E opened a new pull request #26: Fix Typo

2020-03-25 Thread GitBox
WALL-E opened a new pull request #26: Fix Typo URL: https://github.com/apache/skywalking-nginx-lua/pull/26 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [skywalking] mrproliu commented on issue #4555: Correlation protocol implement

2020-03-25 Thread GitBox
mrproliu commented on issue #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555#issuecomment-603820006 > It was best to commit one pull request for one feature This pull request is only used to provide the Java implementation of the correlation

[GitHub] [skywalking] kezhenxu94 commented on issue #4555: Correlation protocol implement

2020-03-25 Thread GitBox
kezhenxu94 commented on issue #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555#issuecomment-603861737 > It was best to commit one pull request for one feature @geektcp It's sure better, but never mind, we do a squash when merging

[GitHub] [skywalking] wu-sheng commented on a change in pull request #4555: Correlation protocol implement

2020-03-25 Thread GitBox
wu-sheng commented on a change in pull request #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555#discussion_r397918574 ## File path: docs/en/setup/service-agent/java-agent/Application-toolkit-trace.md ## @@ -53,3 +53,14 @@ public

[GitHub] [skywalking] wu-sheng commented on a change in pull request #4555: Correlation protocol implement

2020-03-25 Thread GitBox
wu-sheng commented on a change in pull request #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555#discussion_r397918749 ## File path: docs/en/setup/service-agent/java-agent/Application-toolkit-trace.md ## @@ -53,3 +53,14 @@ public

[GitHub] [skywalking] mrproliu commented on a change in pull request #4555: Correlation protocol implement

2020-03-25 Thread GitBox
mrproliu commented on a change in pull request #4555: Correlation protocol implement URL: https://github.com/apache/skywalking/pull/4555#discussion_r397924073 ## File path: docs/en/setup/service-agent/java-agent/Application-toolkit-trace.md ## @@ -53,3 +53,14 @@ public

[skywalking] branch remove-endpoint-register updated: Step 3. Change all source codes of backend, and make project compiling successfully.

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git The following commit(s) were added to refs/heads/remove-endpoint-register by this push: new

[GitHub] [skywalking] hanahmily opened a new issue #4569: Make telemetry UID of kubernetes coordination human-readable

2020-03-25 Thread GitBox
hanahmily opened a new issue #4569: Make telemetry UID of kubernetes coordination human-readable URL: https://github.com/apache/skywalking/issues/4569 Some users find that some Prometheus metrics' instance label is the Pod UUID when they use Kubernetes coordination. We should

[GitHub] [skywalking-kubernetes] innerpeacez edited a comment on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez edited a comment on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603717802 Already feedback.And I found that the `legacy-helm-chart` branch is gone. I don't know why,and I do n’t remember I deleted it. @wu-sheng

[GitHub] [skywalking-kubernetes] innerpeacez commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603722841 > As those are not Apache release, this is not a big issue. But please remove those words from the readme to avoid mislead. I still have this branch

[skywalking-kubernetes] 01/01: legacy-helm-chart branch

2020-03-25 Thread innerpeacez
This is an automated email from the ASF dual-hosted git repository. innerpeacez pushed a commit to branch legacy-helm-chart in repository https://gitbox.apache.org/repos/asf/skywalking-kubernetes.git commit 32db90dfa3aa7e44e2dcc5aa042a68bf1cddcddb Author: innerpeacez AuthorDate: Mon Nov 18

[GitHub] [skywalking] wu-sheng merged pull request #4568: Update How-to-build.md

2020-03-25 Thread GitBox
wu-sheng merged pull request #4568: Update How-to-build.md URL: https://github.com/apache/skywalking/pull/4568 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[skywalking] branch master updated: Update How-to-build.md (#4568)

2020-03-25 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 d53b27d Update How-to-build.md (#4568)

[skywalking-kubernetes] branch legacy-helm-chart created (now 32db90d)

2020-03-25 Thread innerpeacez
This is an automated email from the ASF dual-hosted git repository. innerpeacez pushed a change to branch legacy-helm-chart in repository https://gitbox.apache.org/repos/asf/skywalking-kubernetes.git. at 32db90d legacy-helm-chart branch This branch includes the following new commits:

[GitHub] [skywalking] wu-sheng commented on a change in pull request #4568: Update How-to-build.md

2020-03-25 Thread GitBox
wu-sheng commented on a change in pull request #4568: Update How-to-build.md URL: https://github.com/apache/skywalking/pull/4568#discussion_r397691096 ## File path: docs/en/guides/How-to-build.md ## @@ -89,7 +89,7 @@ Refer to [Build docker image](../../../docker) for more

[GitHub] [skywalking-kubernetes] innerpeacez commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603728379 Done.The strange thing is that the reply before `infra` was that both the master branch and the legacy-helm-chart branch are protected

[GitHub] [skywalking-kubernetes] innerpeacez commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603717802 Already feedback.And I found that the `legacy-helm-chart` branch is gone. I don't know why,and I do n’t remember I deleted it.

[GitHub] [skywalking] wu-sheng commented on a change in pull request #4568: Update How-to-build.md

2020-03-25 Thread GitBox
wu-sheng commented on a change in pull request #4568: Update How-to-build.md URL: https://github.com/apache/skywalking/pull/4568#discussion_r397691096 ## File path: docs/en/guides/How-to-build.md ## @@ -89,7 +89,7 @@ Refer to [Build docker image](../../../docker) for more

[GitHub] [skywalking-kubernetes] wu-sheng commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
wu-sheng commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603721355 > Already feedback.And I found that the legacy-helm-chart branch is gone. I don't know why,and I do n’t remember I deleted it. Yes. Super strange. I

[GitHub] [skywalking-kubernetes] wu-sheng commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
wu-sheng commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603721945 As those are not Apache release, this is not a big issue. But please remove those words from the readme to avoid mislead.

[GitHub] [skywalking-kubernetes] wu-sheng commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
wu-sheng commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603725194 > I still have this branch locally, resubmit this branch? You can, and request the branch protection this time.

[GitHub] [skywalking-kubernetes] innerpeacez edited a comment on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez edited a comment on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603728379 Done.The strange thing is that the reply from `infra` was that both the master branch and the legacy-helm-chart branch are protected

[GitHub] [skywalking-kubernetes] innerpeacez opened a new pull request #47: 2.0.0 change

2020-03-25 Thread GitBox
innerpeacez opened a new pull request #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [skywalking] songzhendong opened a new pull request #4568: Update How-to-build.md

2020-03-25 Thread GitBox
songzhendong opened a new pull request #4568: Update How-to-build.md URL: https://github.com/apache/skywalking/pull/4568 Modify Eclipse and IDEA steps Please answer these questions before submitting pull request - Why submit this pull request? - [ ] Bug fix - [ ] New

[GitHub] [skywalking] codecov-io commented on issue #4568: Update How-to-build.md

2020-03-25 Thread GitBox
codecov-io commented on issue #4568: Update How-to-build.md URL: https://github.com/apache/skywalking/pull/4568#issuecomment-603687878 # [Codecov](https://codecov.io/gh/apache/skywalking/pull/4568?src=pr=h1) Report > Merging

[skywalking] branch remove-endpoint-register updated: Step 2, finish the basic and core DAO changes.

2020-03-25 Thread wusheng
This is an automated email from the ASF dual-hosted git repository. wusheng pushed a commit to branch remove-endpoint-register in repository https://gitbox.apache.org/repos/asf/skywalking.git The following commit(s) were added to refs/heads/remove-endpoint-register by this push: new

[GitHub] [skywalking] codecov-io edited a comment on issue #4517: fix elasticsearch-5.x-plugin when use es6.x TransportClient error

2020-03-25 Thread GitBox
codecov-io edited a comment on issue #4517: fix elasticsearch-5.x-plugin when use es6.x TransportClient error URL: https://github.com/apache/skywalking/pull/4517#issuecomment-600967432 # [Codecov](https://codecov.io/gh/apache/skywalking/pull/4517?src=pr=h1) Report > Merging

[GitHub] [skywalking-kubernetes] wu-sheng commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
wu-sheng commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603713177 @innerpeacez Do you submit the wrong request to INFRA? It requires `.github/workflows/chart-deploy-ci.yaml ` task. We can't merge the PR untill INFRA changed

[GitHub] [skywalking-kubernetes] wu-sheng commented on issue #47: 2.0.0 change

2020-03-25 Thread GitBox
wu-sheng commented on issue #47: 2.0.0 change URL: https://github.com/apache/skywalking-kubernetes/pull/47#issuecomment-603799816 If it is protected, ask Infra about this. We can't delete once it is protected. This is an