Uploaded image for project: 'Sentry (Retired)'
  1. Sentry (Retired)
  2. SENTRY-1481

Evaluate use of paths in MAuthzPathsMapping

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 2.0.0
    • None
    • None
    • None

    Description

      In SentryStore#retrieveFullPathsImage, I see that we essentially loop over all rows of MAuthzPathsMapping just to create a slightly different representation of it. This might be pretty expensive as this table can be huge? I think we should consider returning MAuthzPathsMapping instead.

      And in equals() of MAuthzPathsMapping, seems like we do not compare paths. Just want to make sure it is by design.
      I see that we are essentially treating this object as a map on authzObjName in application logic.

      Attachments

        Issue Links

          is related to

          New Feature - A new feature of the product, which has yet to be developed. SENTRY-872 Uber jira for HMS HA + Sentry HA redesign

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Activity

            People

              btowles Brian Towles
              sravya Sravya Tirukkovalur
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: