Uploaded image for project: 'Sentry'
  1. Sentry
  2. SENTRY-1031

Reorganize sentry mvn modules

    XMLWordPrintableJSON

    Details

    • Type: Task
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Build, Service
    • Labels:
      None

      Description

      See the review for SENTRY-995 (https://reviews.apache.org/r/42682/) for context.

      Some issues/ideas:

      • sentry-provider-db is confusingly named (db refers to database-backed, but it's confusing when there is also a database model). Rename it to something service related?
      • sentry-provider-db implies it is provider related (i.e. will be used by external services), but it has code that an external service would never use (e.g. shells). Perhaps break it up so that sentry-client is a separate top level module which sentry-provider-db pulls in and contains the shell code, or just move out the shell code to a sentry-shell top level module.
      • External services have code scattered through the modules. For example, for solr there is a the sentry-binding-solr, the solr-sentry code, and the shell code (as well as test code, but that's probably okay) all in separate modules. It would be ideal if all the external service specific code was in a single place.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                gchanan Gregory Chanan
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: