You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/09/24 15:44:56 UTC

[GitHub] [superset] yongchand opened a new issue #16834: Mixed Time-Series chart rendering issue

yongchand opened a new issue #16834:
URL: https://github.com/apache/superset/issues/16834


   Unexpected error shows when rendering chart occasionally. 
   
   ### Expected results
   
   Chart should show the rendering result
   
   ### Actual results
   
   Occasionally, chart fails on rendering and shows "An error occurred while rendering the visualization: TypeError: Cannot read properties of undefined (reading 'data')"
   
   #### Screenshots
   
   <img width="1120" alt="Screen Shot 2021-09-24 at 8 03 26 PM" src="https://user-images.githubusercontent.com/43336767/134702697-62a247f6-c52f-4b2d-a0e5-b603e121188b.png">
   
   #### How to reproduce the bug
   
   1. Update to Superset 1.3 (specific commit_sha: 6d19de8f66e6409cc82d6d1e5db2496c1ca2290d)
   2. Create a mixed timeseries chart
   3. Render multiple times and check error 
   
   ### Environment
   
   (please complete the following information):
   
   - browser type and version: Chrome 93
   - superset version: 1.3
   - python version: `python --version`
   - node.js version: `node -v`
   - any feature flags active: GLOBAL_ASYNC_QUERIES
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
   - [ ] I have reproduced the issue with at least the latest released version of superset.
   - [ ] I have checked the issue tracker for the same issue and I haven't found one similar.
   
   ### Additional context
   
   Add any other context about the problem here.
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995


   @junlincc So based on what we have found out until now, we found out that if cache does not exist, rendering mixed timeseries was totally fine. If cache exist, we found out that this error was reproduced. As explained on #15877 , we have turned on GLOBAL ASYNC QUERIES. 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-931479428


   My coworker @yougyoung94 might know how to solve this issue


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yougyoung94 commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yougyoung94 commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-931840738


   @suddjian I opened a PR:):) Waiting for your review 🙏 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995






-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] villebro closed issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
villebro closed issue #16834:
URL: https://github.com/apache/superset/issues/16834


   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995


   @junlincc So based on what we have found out until now, we found out that if cache does not exist, rendering mixed timeseries was totally fine. If cache exist, we found out that this error was reproduced, calling different api endpoint as @rwspielman explained on #15877. We have turned on GLOBAL ASYNC QUERIES. When we did not turned on   GLOBAL ASYNC QUERIES from config file, this error was not reproduced. We used most recent version of 1.3.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] junlincc commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
junlincc commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928873083


   hi! what do you mean by "rendering multiple times"?  is it only happening to said chart? can you reproduce it constantly? thanks! 
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995


   @junlincc So based on what we have found out until now, we found out that if cache does not exist, rendering mixed timeseries was totally fine. If cache exist, we found out that this error was reproduced, calling different api endpoint as @rwspielman explained on #15877. We have turned on GLOBAL ASYNC QUERIES. When we did turned off GLOBAL ASYNC QUERIES from config file, the error was not reproduced. We used most recent version of 1.3.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] junlincc edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
junlincc edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-929585437






-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995


   @junlincc So based on what we have found out until now, we found out that if cache does not exist, rendering mixed timeseries was totally fine. If cache exist, we found out that this error was reproduced, calling different api endpoint as @rwspielman explained on #15877. We have turned on GLOBAL ASYNC QUERIES. When we did not turned on   GLOBAL ASYNC QUERIES from config file, this error was not reproduced. 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] junlincc commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
junlincc commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928873083


   hi! what do you mean by "rendering multiple times"?  is it only happening to said chart? can you reproduce it constantly? thanks! 
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] junlincc commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
junlincc commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-929585437


   Gotcha, thanks for reporting! another feature flag conflict 🤦🏾‍♀️
   @villebro can you walk @stephenLYZ through the process of adding the support? Thanks! 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] yongchand commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
yongchand commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-928875995


   @junlincc So based on what we have found out until now, we found out that if cache does not exist, rendering mixed timeseries was totally fine. If cache exist, we found out that this error was reproduced. As explained on #15877 , we have turned on GLOBAL ASYNC QUERIES. 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-931746165


   @yongchand Happy to collab or review a PR


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #16834: Mixed Time-Series chart rendering issue

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #16834:
URL: https://github.com/apache/superset/issues/16834#issuecomment-931746165


   @yougyoung94 Happy to collab or review a PR


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org