[IGNITE-12441] Refactor SystemViewRowAttributeWalker registration for system views

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.9
• Labels: IEP-35
• Assignee: Aleksey Plekhanov
• Reporter: Aleksey Plekhanov
• Created: 13/Dec/19 07:38
• Updated: 22/Apr/20 13:06
• Resolved: 13/Dec/19 14:17

Description

Currently, to register system view we need at least to actions:

1. Register walker for a system view class
2. Register system view itself for a given class

There is a map maintained to bind the attribute walker and system view class.

Sometimes walker registration code and system view registration code are located in different parts of Ignite.

But in most cases, there is 1:1 relation between attribute walkers and system views. We don't need to maintain additional structures and do additional actions to find a walker by system view class, we can bind walker to system view itself and can do it atomically.

Moreover, in the current implementation, there is impossible to use different walkers for one system view class (for example when we want to create two views with the same system view class, but with different columns set)

 

Issue Links

• fixes: IGNITE-12907 GridSystemViewManager is not thread safe
• relates to: IGNITE-12301 Free-lists system view
• links to: GitHub Pull Request #7136