db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Daniel John Debrunner (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-3452) ShutdownException and ErrorStringBuilder classes do not belong to the services context module
Date Thu, 28 Feb 2008 22:47:51 GMT

    [ https://issues.apache.org/jira/browse/DERBY-3452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12573500#action_12573500
] 

Daniel John Debrunner commented on DERBY-3452:
----------------------------------------------

Applied patch Revision: 632129 with minor cleanup to correct the package names in the ASF
licence header for the moved files. Thanks Dibyendu.

I actually think the ErrorStringBuilder belongs in the iapi.services.streams module but in
a slightly different form.

A set of logical operations on InfoStreams that mirrored a sub-set of  java.util.logging.Logger
would provide benefits such as dynamic changing the level of the log information, including
through standard JMX platform MBeans allowing other code to add handlers to Derby's logging
etc.

E.g. warning(), info(), config(), throwing() with the latter replacing the functionality of
ErrorStringBuilder.

I'll enter a new jira issue for this though.

> ShutdownException and ErrorStringBuilder classes do not belong to the services context
module
> ---------------------------------------------------------------------------------------------
>
>                 Key: DERBY-3452
>                 URL: https://issues.apache.org/jira/browse/DERBY-3452
>             Project: Derby
>          Issue Type: Sub-task
>            Reporter: Dibyendu Majumdar
>            Assignee: Dibyendu Majumdar
>         Attachments: DERBY-3452_patch_rev630512.txt
>
>
> The classes ShutdownException and ErrorStringBuilder do not seem well placed in org.apache.derby.iapi.services.context
as they are more about errors than about contexts.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message