Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RFC: Network faults during effect processing can lead to stale state data #153

Open
erhathaway opened this issue Jun 29, 2021 · 3 comments

Comments

@erhathaway
Copy link
Contributor

erhathaway commented Jun 29, 2021

I suspect that network faults during effect processing can lead to stale state data might be an issue. There needs to be some validation if this is in fact a problem.

If it is a problem, I think a few solutions exist:

  • Add retry logic with some sort of back off to effect processing. Expose a hasNetworkFault attribute to indicate that the UI is out of date

or

  • Add a concept of request and response filter state. (This is a large change and dangerous b/c test coverage is very poor!)
    • The ui is driven off the response state
    • Effect requests are driven off the request state
    • Submission of the effects successfully updates filter response states
@erhathaway erhathaway changed the title RFC: Network faults during effect processing can lead to state date RFC: Network faults during effect processing can lead to state data Jun 29, 2021
@erhathaway erhathaway changed the title RFC: Network faults during effect processing can lead to state data RFC: Network faults during effect processing can lead to stale state data Jun 29, 2021
@dearsaturn
Copy link

Steps to reproduce either this or a similar issue:

  1. Add a filter.
  2. Set the column sort to sort by something that is not configured correctly. See that the query fails in the console, but the user is allowed to continue setting additional filters.
  3. Update the filter created in step 1. See that a new query is run, but results do not change.
  4. Note that the error from step 2 is continuing to occur, breaking every query.
  5. Remove the problematic sort.
  6. Note that functionality is restored queries start to succeed again.

@erhathaway
Copy link
Contributor Author

@dearsaturn I think that may actually be a different bug. There is no validation on filter or sort names existing, and I think this is just putting bad data into the system which is breaking all future requests

@erhathaway
Copy link
Contributor Author

@dearsaturn this RFC deals with a potential bug that happens if a network fault occurs mid request during a valid filter / sort

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants