finos / finos/vuu

Vuu server sends redundant extra TABLE_ROWS message

Open
#562 0 comments 0 reactions 0 assignees View on GitHub
server
Dominant language
TypeScript
Stars
58
Forks
47
Avg merge
13h 3m
Merged PRs (30d)
110

Description

When client sends CHANGE_VP_RANGE messages , we expect a CHANGE_VP_RANGE_SUCCESS followed by TABLE_ROWS with rows in the new range (the TABLE_ROWS may be split across multiple messages but they are mostly sent as a singlr batch in one message)

What is actually happening very commonly is three messages are received for every CHANGE_VP_RANGE message sent.
1) The ack CHANGE_VP_RANGE_SUCCESS
2) the table rows TABLE_ROWS
3) a spurious TABLE_ROWS message with a single row - the last row of the preceding message
The pattern can be seen in the first screenshot below. The green messages are CHANGE_VP_RANGE requests, the white messages are server responses.

![Screenshot 2023-03-05 at 09 23 02](https://user-images.githubusercontent.com/40262/222952322-03f66d49-f724-47b7-b3d0-35ee2dadb248.png)

The following screenshots show the last row from the first TABLE_ROWS message, followed by the same row repeated in the following TABLE_ROWS message, just to illustrate they the data is the same

![Screenshot 2023-03-05 at 09 28 03](https://user-images.githubusercontent.com/40262/222952556-77cf57b3-7f15-4351-83fe-27b7ed482b99.png)
![Screenshot 2023-03-05 at 09 28 20](https://user-images.githubusercontent.com/40262/222952558-6b4cf6d5-5f80-43ef-942b-fb5132e7381d.png)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.