[GitHub] [superset] spyoungtech commented on issue #24487: SQL Lab UI does not populate results - query/updated_since endpoint unexpectedly returns empty array

["spyoungtech (via GitHub)" <gi...@apache.org>]

spyoungtech commented on issue #24487:
URL: https://github.com/apache/superset/issues/24487#issuecomment-1603385470

   Our current understanding is [this value for `last_updated_ms` is generated by](https://github.com/apache/superset/blob/2a4ef5cccf3d3f8f2f7ccfc8c590f8f2fde31ab0/superset-frontend/src/SqlLab/reducers/sqlLab.js#L617) calling `Date.parse` on the `changed_on` attribute returned from the call to the `/sqllab/execute` endpoint. For example, one such value returned was (accurately) `"changed_on": "2023-06-22T21:52:29.488110"`. However, then this value is run through `Date.parse` the returned value is not quite expected:
   
   ```javascript
   Date.parse("2023-06-22T21:52:29.488110")
   1687495949488  // in ISO format: 2023-06-23T04:52:29.488000
   ```
   
   This appears to be the source of the bug. My estimation is that either the parsing method in javascript or format of data returned by the `/sqllab/execute` endpoint needs to be changed to fix this.


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