Add better error logging to perspective manager message handling

finos / perspective

A data visualization and analytics component, especially well-suited for large and/or streaming datasets.

https://perspective.finos.org/

Apache License 2.0

8.36k stars 1.17k forks source link

Add better error logging to perspective manager message handling #2571

Closed timkpaine closed 4 months ago

timkpaine commented 6 months ago

I recently had to debug a nasty issue which ended up being entirely due to some misuse of react. However, in the course of doing so, I noticed some server-side errors being swallowed. This PR adds a bit more verbose logging when client/server errors occur.

It will soon be obsolete due to the port to rust.

texodus commented 5 months ago

Is this still a draft @timkpaine ?

timkpaine commented 5 months ago

@texodus its good to go but technically all this code will go away with the client/server overhaul.

texodus commented 4 months ago

Thanks for the PR!

I've cherry-picked this into #2623, closing.