[GitHub] [druid] vogievetsky opened a new pull request #10741: Web console: fix bad results if there is not native bigint

[GitBox]

vogievetsky opened a new pull request #10741:
URL: https://github.com/apache/druid/pull/10741


   This PR bumps the version of json-bigint-native to fix a bug where it would 
throw an error if `BigInt` was not defined (instead of doing a fallback).
   
   This caused the results not to get parsed (due to the code in 
https://github.com/apache/druid/blob/master/web-console/src/singletons/api.ts#L37)
 and be returned as a string, which causes this madness:
   
   
![image](https://user-images.githubusercontent.com/177816/104140417-4e475780-5366-11eb-9939-73bdcc7ebf27.png)
   
   This is on Safari 13.1 (which does not have `BigInt` yet)
   
   Added a test to make sure there is a graceful fallback to `number`


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org