[jira] Created: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
------------------------------------------------------------------------------------------------------

                 Key: CAMEL-964
                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
             Project: Apache Camel
          Issue Type: Bug
          Components: camel-core
    Affects Versions: 1.5.0
            Reporter: Martin Gilday
         Attachments: logger.patch

Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.

Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:

LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
errorBuilder.setLevel(LoggingLevel.ERROR);
errorHandler(errorBuilder);

instead of:

errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));

I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.



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

Reply | Threaded
Open this post in threaded view
|

[jira] Assigned: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen reassigned CAMEL-964:
---------------------------------

    Assignee: Claus Ibsen

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

    [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=46281#action_46281 ]

Claus Ibsen commented on CAMEL-964:
-----------------------------------

Martin please re-attach the patch with Apache license granted

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen updated CAMEL-964:
------------------------------

       Issue Type: Improvement  (was: Bug)
    Fix Version/s: 1.5.0

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Martin Gilday updated CAMEL-964:
--------------------------------

    Attachment: logger.patch

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch, logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Martin Gilday updated CAMEL-964:
--------------------------------

    Attachment:     (was: logger.patch)

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

    [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=46282#action_46282 ]

Martin Gilday commented on CAMEL-964:
-------------------------------------

Patch reattached.

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Resolved: (CAMEL-964) loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen resolved CAMEL-964.
-------------------------------

    Resolution: Fixed

Applied patch with thanks

> loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-964
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-964
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Martin Gilday
>            Assignee: Claus Ibsen
>             Fix For: 1.5.0
>
>         Attachments: logger.patch
>
>
> Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.
> Also some of the methods on this builder to not return "this".  This breaks the chain when attempting to use the builder resulting in:
> LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
> errorBuilder.setLevel(LoggingLevel.ERROR);
> errorHandler(errorBuilder);
> instead of:
> errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));
> I have added on builder methods for "level" and for "log".  I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set".  Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.

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