Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-595

Using derby.language.logStatementText=true can mask certain exceptions and lead to incorrect behavior in some cases

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.2.1.6, 10.3.1.4
    • 10.2.2.1, 10.3.1.4
    • SQL
    • None
    • all

    Description

      Using derby.language.logStatementText=true can mask certain exceptions and lead to incorrect behavior.

      I observed this with tests using streams, where if valid (expected) exceptions are raised when DVD.getString() is called, the exception gets eaten up when this property is set.

      See
      1)in GenericParameter.toString()
      try
      {
      return value.getString();
      }
      catch (StandardException se)
      {
      return "unexpected exception from getString() - " + se;
      }
      }

      2)in GenericPreparedStatement.execute(), where pvs.toString() is called for the parameters.
      ________

      Reproduction: Run the test jdbcapi/resultsetStream.java , with derby.language.logStatementText=true and expected error exceptions wont be thrown for the error cases.

      I looked at the tests that use streams , only the store/streamingColumn.java uses derby.language.logStatementText=true. I'll file another bug to resolve this test.

      Attachments

        1. derby595.diff
          4 kB
          Mayuresh Nirhali
        2. derby595_v2.diff
          3 kB
          Mayuresh Nirhali

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            mayureshnirhali Mayuresh Nirhali
            skambha Sunitha Kambhampati
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment