You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Bryan Pendleton (JIRA)" <ji...@apache.org> on 2010/04/06 16:10:33 UTC
[jira] Commented: (DERBY-4608) Unnecessary conversion of binary
values to strings in SQLBinary.compare()
[ https://issues.apache.org/jira/browse/DERBY-4608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12853985#action_12853985 ]
Bryan Pendleton commented on DERBY-4608:
----------------------------------------
The patch looks good to me, thanks for tracking this down.
I don't particularly care for the wording of the THROWASSERT
in SQLBinary.java just prior to your change, but of course that's
entirely unrelated, just something that struck my eye while
reading your patch.
+1 to your patch, which looks thorough and complete.
> Unnecessary conversion of binary values to strings in SQLBinary.compare()
> -------------------------------------------------------------------------
>
> Key: DERBY-4608
> URL: https://issues.apache.org/jira/browse/DERBY-4608
> Project: Derby
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 10.5.3.0
> Reporter: Knut Anders Hatlen
> Assignee: Knut Anders Hatlen
> Attachments: isnull.diff
>
>
> SQLBinary.compare(int,DataValueDescriptor,boolean,boolean) converts the values to strings in order to check whether any of them are null. The isNull() method should be used instead to prevent the unnecessary conversion to strings.
> See this thread on derby-user: http://mail-archives.apache.org/mod_mbox/db-derby-user/201003.mbox/%3C001801cad09b$09aef650$1d0ce2f0$@ru%3E
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.