[GitHub] hanahmily commented on issue #775: Feature/graphql ui

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

hanahmily commented on issue #775: Feature/graphql ui
URL: https://github.com/apache/incubator-skywalking/pull/775#issuecomment-362526500
 
 
   Sorry for the late reply. There is one issue for `getApplicationTopology` query.
   
   ## `getAllApplication` query can return some data
   ### query 
   ```graphql
   query ApplicationOption($duration: Duration!) {
      getAllApplication(duration: $duration) {
       key: id
       label: name
     }
   }
   ```
   ### variables
   ```json
   {"duration":{"start":"2018-02-02 1538","end":"2018-02-02 1638","step":"MINUTE"}}
   ```
   
   ### data
   
   ```json
   {
     "data": {
       "getAllApplication": [
         {
           "key": "-2",
           "label": "172.25.0.4:20880"
         },
         {
           "key": "1",
           "label": "User"
         },
         {
           "key": "3",
           "label": "localhost:27017"
         }
       ]
     }
   }
   ```
   
   ## But `getApplicationTopology` query CAN NOT return any data
   ### query 
   ```graphql
   query Application($applicationId: ID!, $duration: Duration!) {
     getApplicationTopology(applicationId: $applicationId, duration: $duration) {
       nodes {
         id
         name
         type
         ... on ApplicationNode {
           sla
           callsPerSec
           responseTimePerSec
           apdex
           isAlarm
           numOfServer
           numOfServerAlarm
           numOfServiceAlarm
         }
       }
       calls {
         source
         target
         isAlert
         callType
         callsPerSec
         responseTimePerSec
       }
     }
   }
   
   ```
   ### variables
   ```json
   {"duration":{"start":"2018-02-02 1538","end":"2018-02-02 1638","step":"MINUTE"},"applicationId":"3"}
   ```
   
   ### data
   
   ```json
   {
     "data": {
       "getApplicationTopology": {
         "nodes": [],
         "calls": []
       }
     }
   }
   ```
   
   I also confirm the arguments of `org.apache.skywalking.apm.collector.ui.service.ApplicationTopologyService#getApplicationTopology` are correct.
   
   Looking forward to @peng-yongsheng reply
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services