gmane.comp.java.mule.devel

Re: [mule-scm] [mule][24202] branches/mule-3.1.x/modules/launcher/src/main/java/org/mule/ module/launcher/descriptor/PropertiesDescriptorParser.java: MULE-6082: Application Deployment Descriptor is not properly closed

Dirk Olmes — 2012-04-21T03:07:08


Shouldn't we better use PropertiesUtils here?

-dirk

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

Re: [mule-scm] [mule][24255] branches/mule-3.2.2-DM/distributions: update log4j version in whitelists

Andrew Perepelytsya — 2012-04-14T00:03:40

That looks safe, thank you :)
On Apr 13, 2012 7:07 PM, "Santiago Vacas" <santiago.vacas-569aJbr3hvNWk0Htik3J/w< at >public.gmane.org>
wrote:

Re: [mule-scm] [mule][24255] branches/mule-3.2.2-DM/distributions: update log4j version in whitelists

Santiago Vacas — 2012-04-13T23:07:21

I don't see that we're overriding any log4j classes in the logging
module, only clogging and slf4j classes.

The logging module shades the following jars:

org.slf4j:jcl-over-slf4j
org.slf4j:slf4j-log4j12
org.slf4j:slf4j-api

according to slf4j docs, org.slf4j:slf4j-log4j12 should work with
log4j version 1.2.12 or later.

http://www.slf4j.org/codes.html#log4j_version

Let me know if I'm missing something.

On Fri, Apr 13, 2012 at 6:52 PM, Andrew Perepelytsya <aperepel-Re5JQEeQqe8AvxtiuMwx3w< at >public.gmane.org> wrote:

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

Re: [mule-scm] [mule][24255] branches/mule-3.2.2-DM/distributions: update log4j version in whitelists

Andrew Perepelytsya — 2012-04-13T21:52:46

Please double-check the change log for this release of log4j. Even better,
do a source diff on log4j files between 1.2.14 and 1.2.16 releases for
files we had to override in Mule. See mule-module-logging sources for this
file list.

Andrew
On Apr 13, 2012 2:22 PM, <svacas-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

Re: [mule-scm] [mule][24244] branches/mule-3.2.x/core/src/test/java/org/mule/routing/ IdempotentMessageFilterMule6079TestCase.java: MULE-6079: IdempotentMessageFilter can process a given message multiple times

Pablo Kraan — 2012-04-12T15:22:09

Hi Saveli,

Some comments:

_ Fix code style: imports, spacing, etc (there are settings for
Eclipse/IntelliJ in the docs)
_ Review used names for variables and classes: endpoint1,
ir, res, ObjectStoreWithLatch, etc. Use more descriptive names.
_ ObjectStoreWithLatch: you don't need to implement the full ObjectStore
contract: it has to implement only the logic required for your test. For
example: don't check for null key in the OS methods and delete the code
inside the remove method if it not used.
_ The call to the fail method inside the thread class is useless because is
not running in the main thread. (same for the usages of fail on the
ObjectStore)
_ The error message in the test's assertEquals is not correct as the
failure could be because there were none or many message processed
_ Replace < at >Test //test for MULE-6079 with  < at >Test("Related to MULE-6079")

Pablo


On Wed, Apr 11, 2012 at 9:18 AM, <saveli.vassiliev-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

Re: [mule-scm] [mule][24166] branches/mule-3.x: adding wildcard support to basic functionality elements

Daniel Feist — 2012-03-25T00:58:07

Shouldn't we reuse org.mule.routing.filters.WildcardFilter (or abstract logic in WildcardFilter into something common), rather than having this in two places?


On Mar 24, 2012, at 8:55 PM, pablo.lagreca-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org wrote:

Re: [mule-scm] [mule][24114] branches/mule-3.x/core/src/main/java/org/mule/processor: Change DefaultMuleEvent.isTransacted() to return null if there' s a transaction in context

Daniel Feist — 2012-03-21T18:45:18

There might be other places where you need to add event.isTransacted();

I cleaned up up all of this, when i put everything inside the resolution of sync in DefaultMuleEvent.

Dan


On Mar 21, 2012, at 3:36 PM, pablo.lagreca-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org wrote:

Re: [mule-scm] [mule][24064] branches/mule-3.x/core/src: MULE-6126 Provide a regex() function that can be used instead of RegexExpressionEvaluator

Pablo Kraan — 2012-03-15T12:35:33

There are a couple of methods (verifyRegex, vefiryText) that are basically
the same. There should be a common verifyStringParam(name, value) to do
that. Same for the verifyFlags.
Validation is done in the call method instead of in the validateParams
method. Looks like validateParams should not be part of the
ExpressionLanguageFunction interface
Regarding tests, I would add a note about why is needed to double escape
the "/" in the regexs (that should be included in the docs too)

Pablo

On Wed, Mar 14, 2012 at 11:55 PM, <dfeist-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

Re: [mule-scm] [mule][23970] branches/mule-3.x: adding causeBy and causeExactlyBy to messaging exception

Pablo La Greca — 2012-03-12T18:06:38

Done!.

On Wed, Mar 7, 2012 at 7:47 PM, Pablo Kraan <pablo.kraan-569aJbr3hvNWk0Htik3J/w< at >public.gmane.org>wrote:

Re: [mule-scm] [mule][23991] branches/mule-3.x/examples/echo: MULE-6117: new echo example

Pablo Kraan — 2012-03-10T22:31:14

Please update the test to use a dynamic port.
Add final to the constant definition.
Remove the < at >author tag

Pablo

On Fri, Mar 9, 2012 at 6:40 PM, <asequeira-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

Re: [mule-scm] [mule][23970] branches/mule-3.x: adding causeBy and causeExactlyBy to messaging exception

Pablo Kraan — 2012-03-07T22:47:32

Looks like MessagingExceptionTestCase can be a candidate for a SmallTest,
can you check that?

Pablo

On Wed, Mar 7, 2012 at 7:00 PM, Daniel Feist <dfeist-Re5JQEeQqe8AvxtiuMwx3w< at >public.gmane.org> wrote:

Re: [mule-scm] [mule][23970] branches/mule-3.x: adding causeBy and causeExactlyBy to messaging exception

Daniel Feist — 2012-03-07T22:00:03

It's not immediately clear what does.  Does it:

- Not look at causes?
- Not look at inherited types?
- Neither?


On Mar 7, 2012, at 6:43 PM, pablo.lagreca-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org wrote:

Re: [mule-scm] [mule][23965] branches/mule-3.x/transports/jdbc: improving SqlStatementStrategy execution to remove duplicate code

Pablo La Greca — 2012-03-07T19:35:19

fixed. Thanks.

On Wed, Mar 7, 2012 at 4:03 PM, Pablo Kraan <pablo.kraan-569aJbr3hvNWk0Htik3J/w< at >public.gmane.org>wrote:

Re: [mule-scm] [mule][23965] branches/mule-3.x/transports/jdbc: improving SqlStatementStrategy execution to remove duplicate code

Pablo Kraan — 2012-03-07T19:03:50

minor things:
_Missing header in the new test file
_Test classes must be named with the TestCase suffix
_Always extend AbstractMuleTestCase in your tests

On Wed, Mar 7, 2012 at 3:49 PM, <pablo.lagreca-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

Re: [3.3][HTTP] HTTP properties usability improvement

Evangelina Martinez — 2012-03-06T14:13:24

Also, if your comment does not include any information about MuleSoft  or
MuleSoft clients please reply to this thread forum:
http://forum.mulesoft.org/mulesoft/topics/_3_3_http_improve_http_properties_usability,
otherwise reply to this email.

Thanks!
Eva

On Tue, Mar 6, 2012 at 11:04 AM, Evangelina Martinez <
evangelina.martinez-569aJbr3hvNWk0Htik3J/w< at >public.gmane.org> wrote:

[3.3][HTTP] HTTP properties usability improvement

Evangelina Martinez — 2012-03-06T14:04:54

Hi,

As part of Mule 3.3 goal to improve the usability this task aims to
facilitate the access to the HTTP properties, review the current properties
and add new ones according to the considered usefulness.

Please see the full specification here:
http://www.mulesoft.org/documentation/display/MULECDEV/HTTP+properties+usability+improvement

This is a first step towards improving the usability in HTTP, the following
task, which specification I will be sending shortly, will focus on
facilitating the creation and configuration of HTTP responses.

Thanks,
Eva

Re: [mule-scm] [mule][23919] branches/mule-3.x/distributions: Fix whitelists for spring security 3.1 upgrade

Mike Schilling — 2012-02-27T20:02:54

It currently seems to be out, but since I have no idea why, I left it 
easy to put back in, should that be required.

Mike

On 2/27/2012 11:37 AM, Daniel Feist wrote:

Re: [mule-scm] [mule][23919] branches/mule-3.x/distributions: Fix whitelists for spring security 3.1 upgrade

Daniel Feist — 2012-02-27T19:37:27

Is aspectjweaver in or out?

Dan

On Feb 27, 2012, at 3:36 PM, mike.schilling-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org wrote:

Re: [mule-scm] [mule][23913] branches/mule-3.x/core/src/test/java/org/mule/routing/ SimpleCollectionAggregatorTestCase.java: MULE-5998 fix

Pablo Kraan — 2012-02-24T15:40:37

Ale,

We add the issue description in the commit message, otherwise we need to
jump to JIRA in order to understand what was fixed.
Review the indentation and the code style of your code.

Pablo


On Fri, Feb 24, 2012 at 12:21 PM, <asequeira-yCVjj/EcxBJg9hUCZPvPmw< at >public.gmane.org> wrote:

[3.3][CORE] Mule Expression Language Spec

Daniel Feist — 2012-02-24T04:04:17

Hi Guys,

I've been working on a spec document for a single expression language for Mule 3.3  (it won't replace expression languages but will live alongside them for now).  Please take a look and let me know what  you think.  Any/all feedback is greatly appreciated.

http://www.mulesoft.org/documentation/display/MULECDEV/Mule+Expression+Language

Let me know if there aren't enough examples and what you would like to see and I can add them.

Also for those that might be interested here is the development branch that has an almost fully functional implementation of this and a distribution built from this branch:

https://svn.codehaus.org/mule/branches/mule-3.x-el
http://dl.dropbox.com/u/1566771/mule-standalone-3.3-M3-EL-SNAPSHOT.zip

thanks!

Dan
---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

Re: Character Encoding

Yuji YAMANO — 2012-02-22T21:55:20

I think it's difficult to answer the quesiton without details,
inbound and outbound transports, config, etc.

On Fri, 17 Feb 2012 05:10:07 -0800 (PST), Narayanan Narayanan <mule.dev.relay-EcxAVUbx2uVoiYX5Tdu9fQ< at >public.gmane.org> wrote: