search

topcoderinc / TC-redis-manager

5 stars 16 forks source link

CLI command and API call optimization #471

Open standlove opened 5 years ago

standlove commented 5 years ago

Now anything we enter in CLI will cause the whole page refreshed.

This is unnecessary. Failed commands and GET commands should NOT cause refresh. And if the command won't affect the tree, no need to fetch the tree data as well.

The point is the front-end should make as less API calls as possible.

standlove commented 5 years ago

Similarly, when user add/update/delete values for list, set, ordered-set, map or string, no need to refresh the tree. You can get the updated-count for the changed key, and set the value to the affected tree node directly.

standlove commented 5 years ago

Actually the tree should only be reloaded in two cases: open the page first time and user clicks the refresh button.

In all other cases, try to update the tree nodes directly.