RE: processing of multipart request
how can I commit code to struts2 SVN repository. any appropriate means and qualify? Link Subject: Re: processing of multipart request From: fastupl...@outlook.com Date: Tue, 13 Nov 2012 10:39:19 +0800 To: dev@struts.apache.org Wes, Struts2 framework uses JakartaMultiPartRequest.java, MultiPartRequest.java, MultiPartRequestWrapper.java to process multipart/form-data request and uses FileUploadInterceptor.java to assemble a File object. At Dispatcher.java line 760 about. the code wraps a multipart/form-data request. request = new MultiPartRequestWrapper(mpr, request, getSaveDir(servletContext)); It leads the leak extension for processing of multipart request. for examples, 1, parsing multipart request into a object contains data of file, it means the data of file store in memory, not on file system. 2, sometime user needs java.io.File in Action, sometime user needs StreamData, struts2 framework should have the ability that provide the user switch easiely. regarding of the requirements, I think the framework may have a factory to create MultiPartRequestWrapper object via configuration specified. or X-work instance MultiPartRequestWrapper object via configuration. Thus, struts2 framework can support varies file upload components, such ASF commons file upload, fast upload and customers upload components. the feature gives a right that user can select to indeed. Best wishes, Link Qian On Nov 12, 2012, at 11:13 PM, Wes Wannemacher w...@wantii.com wrote: Another approach would be for you to take a look at our plugin API and build a plugin for integrating your framework into a Struts 2 web-app. -Wes On Mon, Nov 12, 2012 at 7:39 AM, Fastupload fastupl...@outlook.com wrote: To who maybe concern, I just notice struts2 framework parses multipart/form-data requesting into a temporary file, and marshal a java.io.File object for struts2 action regarding file input of multipart-form data request. also the framework parses all files in request and then make a filter rule in FileUploadInterceptor.java. And finally, framework clean temporary files after action is invoked. Following the processing, there are three disadvantages: 1, use temporary file makes low performance 2, redundancy operation -- clean up temporary file 3, struts action only supports java.io.File class to represent a uploading file in multipart/form-data request. 4, filter all file after parsing has been done, the action may be prior to interceptors. Could you guys have a look into *fastupload* open source project in https://sourceforge.net/projects/fastupload. Right now it is the fastest form-based upload parsing component in java area. it makes a significant performance improvement and advance features, such as, 1, 2.x faster than Apache Commons FileUpload in stream method. certainly, it's 5.x faster than apache commons file upload in disk method. 2, filter boundaries with determining MIME and file name in advance. In fastupload-0.4.7 version, it has released some codes that work with struts2 framework, enable parsing files with it in struts2 framework. To improve features of struts2 multipart/form-data request, Could you consider to integrate *fastupload* source code into struts2 framework. incurring struts2 supports both *fastupload* and ASF commons file upload? best wishes, Link Qian - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org -- Wes Wannemacher Head Engineer, WanTii, Inc. Need Training? Struts, Spring, Maven, Tomcat... Ask me for a quote! - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
RE: processing of multipart request
You'd need to be a committer. In the meantime, filing a jira ticket with a patch is an alternative. Dave (pardon brevity, typos, and top-quoting; on cell) On Nov 14, 2012 8:18 PM, Link Qian fastupl...@outlook.com wrote: how can I commit code to struts2 SVN repository. any appropriate means and qualify? Link Subject: Re: processing of multipart request From: fastupl...@outlook.com Date: Tue, 13 Nov 2012 10:39:19 +0800 To: dev@struts.apache.org Wes, Struts2 framework uses JakartaMultiPartRequest.java, MultiPartRequest.java, MultiPartRequestWrapper.java to process multipart/form-data request and uses FileUploadInterceptor.java to assemble a File object. At Dispatcher.java line 760 about. the code wraps a multipart/form-data request. request = new MultiPartRequestWrapper(mpr, request, getSaveDir(servletContext)); It leads the leak extension for processing of multipart request. for examples, 1, parsing multipart request into a object contains data of file, it means the data of file store in memory, not on file system. 2, sometime user needs java.io.File in Action, sometime user needs StreamData, struts2 framework should have the ability that provide the user switch easiely. regarding of the requirements, I think the framework may have a factory to create MultiPartRequestWrapper object via configuration specified. or X-work instance MultiPartRequestWrapper object via configuration. Thus, struts2 framework can support varies file upload components, such ASF commons file upload, fast upload and customers upload components. the feature gives a right that user can select to indeed. Best wishes, Link Qian On Nov 12, 2012, at 11:13 PM, Wes Wannemacher w...@wantii.com wrote: Another approach would be for you to take a look at our plugin API and build a plugin for integrating your framework into a Struts 2 web-app. -Wes On Mon, Nov 12, 2012 at 7:39 AM, Fastupload fastupl...@outlook.com wrote: To who maybe concern, I just notice struts2 framework parses multipart/form-data requesting into a temporary file, and marshal a java.io.File object for struts2 action regarding file input of multipart-form data request. also the framework parses all files in request and then make a filter rule in FileUploadInterceptor.java. And finally, framework clean temporary files after action is invoked. Following the processing, there are three disadvantages: 1, use temporary file makes low performance 2, redundancy operation -- clean up temporary file 3, struts action only supports java.io.File class to represent a uploading file in multipart/form-data request. 4, filter all file after parsing has been done, the action may be prior to interceptors. Could you guys have a look into *fastupload* open source project in https://sourceforge.net/projects/fastupload. Right now it is the fastest form-based upload parsing component in java area. it makes a significant performance improvement and advance features, such as, 1, 2.x faster than Apache Commons FileUpload in stream method. certainly, it's 5.x faster than apache commons file upload in disk method. 2, filter boundaries with determining MIME and file name in advance. In fastupload-0.4.7 version, it has released some codes that work with struts2 framework, enable parsing files with it in struts2 framework. To improve features of struts2 multipart/form-data request, Could you consider to integrate *fastupload* source code into struts2 framework. incurring struts2 supports both *fastupload* and ASF commons file upload? best wishes, Link Qian - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org -- Wes Wannemacher Head Engineer, WanTii, Inc. Need Training? Struts, Spring, Maven, Tomcat... Ask me for a quote! - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
RE: processing of multipart request
any guide about how filing a JIRA ticket? Link Date: Wed, 14 Nov 2012 20:29:27 -0500 Subject: RE: processing of multipart request From: davelnew...@gmail.com To: dev@struts.apache.org You'd need to be a committer. In the meantime, filing a jira ticket with a patch is an alternative. Dave (pardon brevity, typos, and top-quoting; on cell) On Nov 14, 2012 8:18 PM, Link Qian fastupl...@outlook.com wrote: how can I commit code to struts2 SVN repository. any appropriate means and qualify? Link Subject: Re: processing of multipart request From: fastupl...@outlook.com Date: Tue, 13 Nov 2012 10:39:19 +0800 To: dev@struts.apache.org Wes, Struts2 framework uses JakartaMultiPartRequest.java, MultiPartRequest.java, MultiPartRequestWrapper.java to process multipart/form-data request and uses FileUploadInterceptor.java to assemble a File object. At Dispatcher.java line 760 about. the code wraps a multipart/form-data request. request = new MultiPartRequestWrapper(mpr, request, getSaveDir(servletContext)); It leads the leak extension for processing of multipart request. for examples, 1, parsing multipart request into a object contains data of file, it means the data of file store in memory, not on file system. 2, sometime user needs java.io.File in Action, sometime user needs StreamData, struts2 framework should have the ability that provide the user switch easiely. regarding of the requirements, I think the framework may have a factory to create MultiPartRequestWrapper object via configuration specified. or X-work instance MultiPartRequestWrapper object via configuration. Thus, struts2 framework can support varies file upload components, such ASF commons file upload, fast upload and customers upload components. the feature gives a right that user can select to indeed. Best wishes, Link Qian On Nov 12, 2012, at 11:13 PM, Wes Wannemacher w...@wantii.com wrote: Another approach would be for you to take a look at our plugin API and build a plugin for integrating your framework into a Struts 2 web-app. -Wes On Mon, Nov 12, 2012 at 7:39 AM, Fastupload fastupl...@outlook.com wrote: To who maybe concern, I just notice struts2 framework parses multipart/form-data requesting into a temporary file, and marshal a java.io.File object for struts2 action regarding file input of multipart-form data request. also the framework parses all files in request and then make a filter rule in FileUploadInterceptor.java. And finally, framework clean temporary files after action is invoked. Following the processing, there are three disadvantages: 1, use temporary file makes low performance 2, redundancy operation -- clean up temporary file 3, struts action only supports java.io.File class to represent a uploading file in multipart/form-data request. 4, filter all file after parsing has been done, the action may be prior to interceptors. Could you guys have a look into *fastupload* open source project in https://sourceforge.net/projects/fastupload. Right now it is the fastest form-based upload parsing component in java area. it makes a significant performance improvement and advance features, such as, 1, 2.x faster than Apache Commons FileUpload in stream method. certainly, it's 5.x faster than apache commons file upload in disk method. 2, filter boundaries with determining MIME and file name in advance. In fastupload-0.4.7 version, it has released some codes that work with struts2 framework, enable parsing files with it in struts2 framework. To improve features of struts2 multipart/form-data request, Could you consider to integrate *fastupload* source code into struts2 framework. incurring struts2 supports both *fastupload* and ASF commons file upload? best wishes, Link Qian - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org -- Wes Wannemacher Head Engineer, WanTii, Inc. Need Training? Struts, Spring, Maven, Tomcat... Ask me for a quote! - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: processing of multipart request
On Mon, Nov 12, 2012 at 4:39 AM, Fastupload fastupl...@outlook.com wrote: To who maybe concern, I just notice struts2 framework parses multipart/form-data requesting into a temporary file, and marshal a java.io.File object for struts2 action regarding file input of multipart-form data request. also the framework parses all files in request and then make a filter rule in FileUploadInterceptor.java. And finally, framework clean temporary files after action is invoked. Following the processing, there are three disadvantages: 1, use temporary file makes low performance 2, redundancy operation -- clean up temporary file 3, struts action only supports java.io.File class to represent a uploading file in multipart/form-data request. 4, filter all file after parsing has been done, the action may be prior to interceptors. All of those could be addressed without switching to a new file upload framework, since Commons FileUpload supports them. It would seem to make more sense to me to expand what we have already, rather than throw it out and replace it with something new and untested (i.e. not in widespread Struts production usage). Could you guys have a look into *fastupload* open source project in https://sourceforge.net/projects/fastupload. Right now it is the fastest form-based upload parsing component in java area. it makes a significant performance improvement and advance features, such as, 1, 2.x faster than Apache Commons FileUpload in stream method. certainly, it's 5.x faster than apache commons file upload in disk method. Please provide some links to the benchmarks you've run to measure those things, and the constraints you used when measuring them. 2, filter boundaries with determining MIME and file name in advance. In fastupload-0.4.7 version, it has released some codes that work with struts2 framework, enable parsing files with it in struts2 framework. To improve features of struts2 multipart/form-data request, Could you consider to integrate *fastupload* source code into struts2 framework. incurring struts2 supports both *fastupload* and ASF commons file upload? I can't find any reference to Struts in the codebase you provided a link to. Perhaps you could provide a reference to the Struts related code? The right approach to integration is basically what Wes originally suggested. It should be usable as a plugin. If changes need to be made to Struts to support that, you can propose those changes for consideration. If they're accepted, then people will be able to choose your fastupload as an alternative to what's there today, if they choose to do so for their application. By the way, your code is hard to check out because of the messed up repository structure, and there are also several typos even in the interface and method names. Those things detract from any confidence in the quality of your code, so you might want to clean them up. -- Martin Cooper best wishes, Link Qian - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org