Uploaded image for project: 'Crucible'
  1. Crucible
  2. CRUC-4998

ChangesetTooltipAction.findChangeSet() eagerly creates cruRevisions

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: High High
    • 2.4.3
    • 2.4.2
    • None

      This means that a simple mouseover on a large ChangeSet can bring Crucible to its knees, because for an N-revision changeset, up to N cruRevisions and N storedPaths are added to CRU_DB through the singleton Hibernate session, blocking all review creation until completed.

      Consider for a 2.4.x release

      Suggested solution:

      1. remove the revision rollout on the tooltip - at most we want to know how many revisions?
      2. audit the subclasses of Source - if RepositorySource is the only changesetCapable subclass, consider avoiding source altogether and asking FishEye directly for this info (careful consideration of cs tooltip support in Crucible-only needed here)
      3. add either a changeset revision counting method to Source,documenting a contract of speedy return. Or add a getChangesetDescription() that returns a ChangeSetDescription object providing summary data?
      4. Bonus refactor: rename Source to RevisionProvider or something. Don't forget subclasses.

            sruiz Seb Ruiz (Inactive)
            bhumphreys BrendanA
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - Not Specified
                Not Specified
                Logged:
                Time Spent - 23m
                23m