You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@skywalking.apache.org by ha...@apache.org on 2018/03/08 08:47:53 UTC

[incubator-skywalking-ui] 01/02: Fix #141 render dependency map error

This is an automated email from the ASF dual-hosted git repository.

hanahmily pushed a commit to branch bug/e2e
in repository https://gitbox.apache.org/repos/asf/incubator-skywalking-ui.git

commit 74a7d5ddab0ef2e82f53545e93e95380c4a7742e
Author: hanahmily <ha...@gmail.com>
AuthorDate: Thu Mar 8 15:54:07 2018 +0800

    Fix #141 render dependency map error
---
 mock/service.js               | 4 ++--
 src/routes/Service/Service.js | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/mock/service.js b/mock/service.js
index 1f1c42d..881d6db 100644
--- a/mock/service.js
+++ b/mock/service.js
@@ -87,8 +87,8 @@ export default {
               target: 1,
               'isAlert|1': true,
               'callType|1': ['rpc', 'http', 'dubbo'],
-              'callsPerSec|0-100': 1,
-              'avgResponseTime|500-5000': 1,
+              'callsPerSec|0-1000': 1,
+              'avgResponseTime|500-5000': 0,
             }))).concat(downNodes.nodes.map(node => (mockjs.mock({
               source: 1,
               target: node.id,
diff --git a/src/routes/Service/Service.js b/src/routes/Service/Service.js
index 4b109b8..0de120e 100644
--- a/src/routes/Service/Service.js
+++ b/src/routes/Service/Service.js
@@ -64,6 +64,7 @@ export default class Service extends PureComponent {
       payload: { variables },
     });
   }
+  edgeWith = edge => edge.callsPerSec * edge.avgResponseTime;
   renderSankey = (data) => {
     if (data.nodes.length < 2) {
       return <span style={{ display: 'none' }} />;
@@ -75,7 +76,7 @@ export default class Service extends PureComponent {
     const nData = {
       nodes: data.nodes,
       edges: data.calls.map(_ =>
-        ({ ..._, value: (_.callsPerSec < 1 ? 1000 : _.callsPerSec * _.avgResponseTime), source: nodesMap.get(`${_.source}`), target: nodesMap.get(`${_.target}`) })),
+        ({ ..._, value: (this.edgeWith(_) < 1 ? 1000 : this.edgeWith(_)), source: nodesMap.get(`${_.source}`), target: nodesMap.get(`${_.target}`) })),
     };
     return (
       <Row gutter={24}>

-- 
To stop receiving notification emails like this one, please contact
hanahmily@apache.org.