You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "Ivan-Shindov (via GitHub)" <gi...@apache.org> on 2024/03/14 09:50:08 UTC

[I] Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits) [superset]

Ivan-Shindov opened a new issue, #27510:
URL: https://github.com/apache/superset/issues/27510

   ### Bug description
   
   Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits)
   
   ### How to reproduce the bug
   
   1. Create a SQL virtual dataset that returns 2 BigInt columns : 1 with less digits(e.g. 5) and 1 with more digits(e.g. 16)
   (It doesn't matter if you cast them explicitly in the SQL or not)
   2. Create a chart out of the dataset
   3. Create a Dashboard and use the chart
   ![Cause](https://github.com/apache/superset/assets/91035897/6674c267-e7bb-4fdb-9b38-876ba455729a)
   4. Add filter using the dataset of type Value and use the column with the long ID and Save
   5. When ready click on the filter and move your cursor
   
   The filter then breaks with React error: https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object%20with%20keys%20%7Bs%2C%20e%2C%20c%7D&args[]=
   
   ![Problem](https://github.com/apache/superset/assets/91035897/de7a3119-87fb-4c80-9afc-f12f624233c8)
   
   ### Screenshots/recordings
   
   ![Problem](https://github.com/apache/superset/assets/91035897/de7a3119-87fb-4c80-9afc-f12f624233c8)
   ![Cause](https://github.com/apache/superset/assets/91035897/6674c267-e7bb-4fdb-9b38-876ba455729a)
   
   
   ### Superset version
   
   3.1.1
   
   ### Python version
   
   3.9
   
   ### Node version
   
   16
   
   ### Browser
   
   Chrome
   
   ### Additional context
   
   _No response_
   
   ### Checklist
   
   - [X] I have searched Superset docs and Slack and didn't find a solution to my problem.
   - [X] I have searched the GitHub issue tracker and didn't find a similar bug report.
   - [X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.


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


Re: [I] Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits) [superset]

Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas closed issue #27510: Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits)
URL: https://github.com/apache/superset/issues/27510


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


Re: [I] Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits) [superset]

Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #27510:
URL: https://github.com/apache/superset/issues/27510#issuecomment-2258687806

   This is still happening on the latest `master` and in 4.1. 
   
   For anyone else looking to repro, you can create a dataset like so
   
   ```
   SELECT
       CAST(12345 AS BIGINT) AS short_value,
       CAST(1234567890125123456 AS BIGINT) AS long_value;
   ```


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


Re: [I] Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits) [superset]

Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #27510:
URL: https://github.com/apache/superset/issues/27510#issuecomment-2770660657

   This is still happening today. I feel like @justinpark did something around bigint support not so long ago... not sure if this will affect his team.


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


Re: [I] Using Filter per value of Column of type BIGINT breaks UI when the value is long(16 digits) [superset]

Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #27510:
URL: https://github.com/apache/superset/issues/27510#issuecomment-4035399630

   Confirmed still present per @rusackas — filter values for BIGINT columns with 16+ digits cause UI issues. The fix likely involves treating these as strings in the filter UI rather than numbers. @justinpark has done related bigint work — any input on this would be appreciated. PRs welcome.


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