A lot of good educations here! Thanks!
The reason why I am making a copy of `this.props.payload.data.features`, 
mutating it, and then assigning it back to payload in line 107 is that if I 
understand correctly, changing in `this.props` will result in re-render. As we 
are only passing it to `getLayer` function and don't need it to be re-rendered, 
I copy it so `props` is unaffected and unnecessary re-render can be avoided. 
This is my understanding but I am not very sure whether it is correct based on 
your suggestion. Please correct me if I am wrong.

[ Full content available at: 
https://github.com/apache/incubator-superset/pull/5767 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to