windhamwong opened a new pull request #3260: Heron API/Tracker update for 
storing Topologies in Array/List instead of Dict object
URL: https://github.com/apache/incubator-heron/pull/3260
 
 
   Old version:
   ```
   {
     (..),
     "result": 
     <cluster>: {
       <environment>: {
         <topology>: {
           (topology_details)
         }
       }
     }
   }
   ```
   New version:
   ```
   {
     (..),
     "result": [{
       (topology_details)
     }]
   }
   ```
   
   The cluster, environment, topology variables are actually obtainable from 
the topology details. This PR is to store these data into array not in dict 
object so there will be less nested object inside and duplicated 
cluster/environment/topology can be also displayed on the UI. (Might be useful 
for multi-cluster with same configurations across all the clusters)
   
   Still need some testing and recommendation. It works perfectly for me, but 
there are minor 500 errors (unrelated to this change) from the original source 
code might need to solve as well. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to