[#MULE-3065] Review all transformers for streaming compatibility

Mule

Review all transformers for streaming compatibility

Created: 04/Mar/08 07:51 PM Updated: 10/Apr/08 06:44 PM

Component/s:

Core: Transformers, Core: Streaming

Affects Version/s:

None

Fix Version/s:

2.0.1

Time Tracking:

Not Specified

Issue Links:

Related

This issue relates to:

MULE-3213 Some transformers that return byte[] should be returning Stream

~~MULE-3055~~ Errorhandler example fails with exception

Labels:
User impact:	Medium

Description

« Hide

All transformers that can receive byte[] should also be able to receiver an InputStream. If this is not the case when streaming is enabled, transformers don't match and are skipped and EntryPointResolver errors occur.

Should all transforms that can receive a byte[] and return a byte[], be returning a Stream if that's what that receive?

IMPORTANT: Ensure that all transformers that receive and use a Stream correctly close the stream in a finally block.

All

Comments

Work Log

Change History

Transitions

FishEye

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Dan Diephouse - 05/Mar/08 12:30 PM

#1: Not required, but that would be very beneficial for users. Otherwise they're going to have to insert their own transformer for a lot of scenarios.

#2: I don't think so. Transformers should return whatever is the most efficient format. So if they can return a Stream or an OutputHandler they should. Then the next thing down the line will have the best opportunity for efficient conversion to what it needs.

#3: A big YES here

[ Permlink | « Hide ]

Dirk Olmes - 12/Mar/08 12:54 PM

This is done in
http://fisheye.codehaus.org/changelog/mule/?cs=11316 and http://fisheye.codehaus.org/changelog/mule/?cs=11336

Keeping the issue open as I have one test left that fails (GZipTransformerRawBytesTestCase)

[ Permlink | « Hide ]

Dirk Olmes - 12/Mar/08 03:16 PM

fixed the failing test in http://fisheye.codehaus.org/changelog/mule/?cs=11337

[ Permlink | « Hide ]

Daniel Feist - 10/Apr/08 11:19 AM

FileToString transformer should accept InputStream type.

http://www.nabble.com/2.0-FileConnector-question-tp16609060p16609060.html

[ Permlink | « Hide ]

Daniel Feist - 10/Apr/08 06:43 PM

Hopefully this is the last of them, I couldn't find any others anyway.

http://svn.mule.codehaus.org/changelog/mule/?cs=11561