You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Sanket Shah <er...@gmail.com> on 2017/11/28 06:47:08 UTC

Review Request 64111: AMBARI-22527: Error while viewing topology details - Storm View

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64111/
-----------------------------------------------------------

Review request for Ambari, Sumit Mohanty, Sriharsha Chintalapani, and Yusaku Sako.


Bugs: AMBARI-22527
    https://issues.apache.org/jira/browse/AMBARI-22527


Repository: ambari


Description
-------

On the latest build, due to few npm packages being updated dynamically, Storm view is running into below mentioned error:

```
Uncaught Error: Minified React error #130; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=130&args[]=object&args[]=%20Check%20the%20render%20method%20of%20%60e%60. for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at n (vendor.bundle.js:4)
    at a (vendor.bundle.js:6)
    at Object.updateChildren (vendor.bundle.js:2)
    at v._reconcilerUpdateChildren (vendor.bundle.js:3)
    at v._updateChildren (vendor.bundle.js:3)
    at v.updateChildren (vendor.bundle.js:3)
    at v._updateDOMChildren (vendor.bundle.js:2)
    at v.updateComponent (vendor.bundle.js:2)
    at v.receiveComponent (vendor.bundle.js:2)
    at Object.receiveComponent (vendor.bundle.js:2)
```


Diffs
-----

  contrib/views/storm/src/main/resources/ui/app/scripts/containers/ComponentDetailView.jsx 2b3ed69 
  contrib/views/storm/src/main/resources/ui/package.json f9e029b 


Diff: https://reviews.apache.org/r/64111/diff/1/


Testing
-------

Manually ran the mvn clean install and created storm view jar.
Successfully deployed the jar and verified working of storm view.
Clicked on a topology and verified the working of the page since that was the one throwing above error.


Thanks,

Sanket Shah