Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-12441

Refactor SystemViewRowAttributeWalker registration for system views

    XMLWordPrintableJSON

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9
    • Component/s: None
    • Labels:

      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)

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                alex_pl Aleksey Plekhanov
                Reporter:
                alex_pl Aleksey Plekhanov
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 0.5h
                  0.5h