Uploaded image for project: 'Keikai'
  1. Keikai
  2. KEIKAI-178

Remove unnecessary book lock under Spreadsheet.java (Refix for ZSS-618)

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Normal Normal
    • 5.1.0
    • None
    • None
    • Security Level: Jimmy
    • None
    • None

      It seems no need to get a write lock under Spreadsheet#renderProperties(), which is a side effect from ZSS-618.

      And ZSS-618 seems to be fixed in a wrong way to get a lock for ModelEventDispatcher.java as well.

      In this ZSS-618 case, we should only get a lock to protect CellImpl#evalFormula() only for multi-threading and change the synchronized to use a write lock instead to boost performance for ZSS-618.

            jumperchen jumperchen
            jumperchen jumperchen
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: