Derby
  1. Derby
  2. DERBY-4542

Avoid materialization where possible when cloning CollatorSQLClob

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.3.3.0, 10.4.2.0, 10.5.3.0, 10.6.1.0
    • Fix Version/s: 10.6.1.0
    • Component/s: Store
    • Labels:
      None
    • Issue & fix info:
      Repro attached

      Description

      Cloning a CollatorSQLClob results in always materializing the value. Since CLOBs can be very large, this should be avoided. There should now be sufficient functionality to do this, see SQLClob.

      1. ClobCollationTest.java
        4 kB
        Kristian Waagan
      2. derby-4477-workaround.diff
        1 kB
        Kristian Waagan
      3. derby-4542-1a-avoid_materialization.diff
        2 kB
        Kristian Waagan

        Activity

        Hide
        Kristian Waagan added a comment -

        Attached a repro ('ClobCollationTest.java') and a workaround for DERBY-4477 (only needed until that issue has been completed). Compile the repro and run it, or read the comments inside, to get instructions.

        Show
        Kristian Waagan added a comment - Attached a repro ('ClobCollationTest.java') and a workaround for DERBY-4477 (only needed until that issue has been completed). Compile the repro and run it, or read the comments inside, to get instructions.
        Hide
        Kristian Waagan added a comment -

        Attached a fix as patch 1a, which asks SQLClob to clone the value as long as materialization isn't forced.

        Regression tests just started.
        Patch ready for review.

        Show
        Kristian Waagan added a comment - Attached a fix as patch 1a, which asks SQLClob to clone the value as long as materialization isn't forced. Regression tests just started. Patch ready for review.
        Hide
        Kristian Waagan added a comment -

        Regression tests passed.
        Committed patch 1a to trunk with revision 910200.

        Show
        Kristian Waagan added a comment - Regression tests passed. Committed patch 1a to trunk with revision 910200.
        Hide
        Kristian Waagan added a comment -

        Closing the issue.

        Show
        Kristian Waagan added a comment - Closing the issue.

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development