You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wink.apache.org by Jesse A Ramos <jr...@us.ibm.com> on 2010/06/21 16:39:47 UTC
[VOTE] Release Wink 1.1.1 (RC1)
Hi,
I apologize for the delay. Had a few issues to work though.
Now that everything is resolved, I'm calling a vote to release Wink 1.1.1
(RC1)
The Maven staging area is at:
https://repository.apache.org/content/repositories/orgapachewink-003/
The distributions are in:
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
This release is tagged at:
https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
(revision 956599)
The vote will be open here for at least 72 hours.
Guide to testing staged releases:
http://maven.apache.org/guides/development/guide-testing-releases.html
[ ] +1
[ ] +0
[ ] -1
-- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Bryant Luk <br...@gmail.com>.
+1
On Mon, Jun 21, 2010 at 9:39 AM, Jesse A Ramos <jr...@us.ibm.com> wrote:
> Hi,
>
> I apologize for the delay. Had a few issues to work though.
>
> Now that everything is resolved, I'm calling a vote to release Wink 1.1.1
> (RC1)
>
> The Maven staging area is at:
> https://repository.apache.org/content/repositories/orgapachewink-003/
>
> The distributions are in:
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>
> This release is tagged at:
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
> (revision 956599)
>
> The vote will be open here for at least 72 hours.
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> [ ] +1
> [ ] +0
> [ ] -1
>
> -- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jason Dillon <ja...@planet57.com>.
The problem affects 1.1.1, but will probably be fixed in 1.1.2, but I left the fixed for unset, leaving it up to you guys to decide.
BTW, you guys should probably mark the 0.1, 1.0, 1.1 versions in Jira as released.
--jason
On Jun 22, 2010, at 10:00 AM, Jesse A Ramos wrote:
> Thanks, Jason
>
> I went ahead and changed the affected version to 1.1.2 for these issues.
> Let me if that's okay or if they should remain as 1.1.1 issues.
>
> -- Jesse Ramos
>
>
>
> From:
> Jason Dillon <ja...@planet57.com>
> To:
> wink-dev@incubator.apache.org
> Date:
> 06/22/2010 11:38 AM
> Subject:
> Re: [VOTE] Release Wink 1.1.1 (RC1)
> Sent by:
> Jason Dillon <ja...@gmail.com>
>
>
>
> Maven fluff:
>
> https://issues.apache.org/jira/browse/WINK-294 (easy to fix)
>
> SLF4J muck:
>
> https://issues.apache.org/jira/browse/WINK-295 (easy to fix)
>
> General logging rant:
>
> https://issues.apache.org/jira/browse/WINK-296 (more complicated,
> requires thinking)
>
> --jason
>
>
> On Jun 22, 2010, at 9:08 AM, Mike Rheinheimer wrote:
>
>> +1, it all looks good to me.
>>
>> Jason, all good suggestions. I think we can work on these over the
>> next few weeks. We'd love to have Jiras and patches though! Hint
>> hint. Wink wink. :)
>>
>> mike
>>
>>
>> On Mon, Jun 21, 2010 at 1:04 PM, Jesse A Ramos <jr...@us.ibm.com>
> wrote:
>>> Thanks, Jason, for the feedback. I agree that these are items we
> should
>>> address in a future release. Do you mind opening some JIRAs for these
>>> against version 1.1.2?
>>>
>>> -- Jesse Ramos
>>>
>>>
>>>
>>>
>>>
>>> From:
>>> Jason Dillon <ja...@planet57.com>
>>> To:
>>> wink-dev@incubator.apache.org
>>> Date:
>>> 06/21/2010 12:48 PM
>>> Subject:
>>> Re: [VOTE] Release Wink 1.1.1 (RC1)
>>> Sent by:
>>> Jason Dillon <ja...@gmail.com>
>>>
>>>
>>>
>>> There are some missing versions for plugins:
>>>
>>> <snip>
>>> [WARNING] Some problems were encountered while building the effective
>>> model for
> org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
>>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>>> must be unique but found duplicate declaration of plugin
>>> org.apache.maven.plugins:maven-war-plugin @
>>> org.apache.wink:wink:1.1.1-incubating,
>>> /Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for org.apache.wink:wink-common:jar:1.1.1-incubating
>>> [WARNING] 'build.plugins.plugin.version' for
>>> org.apache.felix:maven-bundle-plugin is missing. @
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for org.apache.wink:wink-server:jar:1.1.1-incubating
>>> [WARNING] 'build.plugins.plugin.version' for
>>> org.apache.felix:maven-bundle-plugin is missing. @
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for org.apache.wink:wink-client:jar:1.1.1-incubating
>>> [WARNING] 'build.plugins.plugin.version' for
>>> org.apache.felix:maven-bundle-plugin is missing. @
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for
>>> org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
>>> [WARNING]
> 'dependencies.dependency.(groupId:artifactId:type:classifier)'
>>> must be unique: junit:junit:jar -> duplicate declaration of version (?)
> @
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
>>> [WARNING] 'build.plugins.plugin.version' for
>>> org.apache.maven.plugins:maven-dependency-plugin is missing. @
>>> [WARNING]
>>> [WARNING] Some problems were encountered while building the effective
>>> model for org.apache.wink:wink:pom:1.1.1-incubating
>>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>>> must be unique but found duplicate declaration of plugin
>>> org.apache.maven.plugins:maven-war-plugin @
>>> [WARNING]
>>> [WARNING] It is highly recommended to fix these problems because they
>>> threaten the stability of your build.
>>> [WARNING]
>>> [WARNING] For this reason, future Maven versions might no longer
> support
>>> building such malformed projects.
>>> </snip>
>>>
>>> I also still highly recommend that you upgrade SLF4j to 1.6.0...
>>>
>>> <snip url="http://slf4j.org/news.html">
>>> As of SLF4J version 1.6.0, in the absence of an SLF4J binding,
> slf4j-api
>>> will default to a no-operation implementation discarding all log
> requests.
>>> Thus, instead of throwing an exception, SLF4J will emit a single
> warning
>>> message about the absence of a binding and proceed to discard all log
>>> requests without further protest. See also the relevant sectionin the
> user
>>> manual.
>>> </snip>
>>>
>>> Looks like the logging format is better, though wink still logs way to
>>> much at DEBUG IMO. For example, this looks more like TRACE logging to
> me:
>>>
>>> <snip>
>>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) called
>>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) called
>>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) returning jobs
>>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) encoded path parameters are: [jobs]
>>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) returning [jobs]
>>> </snip>
>>>
>>> Also the PrivodersRegister logging is still *insane* IMO.
>>>
>>> I think most of the DEBUG logging that is going on should really be
> TRACE
>>> logging. For just one simple GET, with DEBUG enabled for
> org.apache.wink,
>>> this gets spat out:
>>>
>>> <snip>
>>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet -
>>> Processing: GET /rest/status (http://localhost:8080/rest/status)
>>>
> [client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>>> Retrieving request processor
>>> org.apache.wink.server.internal.RequestProcessor@6128453c using
> attribute
>>> name org.apache.wink.server.internal.RequestProcessor in servlet
> context
>>>
> ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath requestPath is: /rest/status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath requestPath normalized is: /rest/status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath after context path removed: /rest/status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath isServlet: true
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath requestPath after servlet path removed: /status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> buildRequestPath returning requestPath: status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) returning status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> HttpServletRequest.getHeaders(Content-Type) returned [] so putting into
>>> headers cache
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getRequestHeaderInternal(Content-Type) returning []
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getMediaType() returning null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set
>>> message context and starting request handlers chain: Method: GET, Path:
>>> status, MediaType: null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.SearchResultHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.OptionsMethodHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.FindRootResourceHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) returning status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) encoded path parameters are: [status]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) returning [status]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
> -
>>> Getting URI Info path segments: [status]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
> -
>>> Getting stripped path from segments: status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
> -
>>> Found resource instances: [ResourceRecord: Path: status; ClassMetadata:
>>> Class: class com.sonatype.benson.rest.api.status.StatusResource;
>>> UriTemplateMatcher: Parent: status; URI: status; Matcher:
>>> java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6
>>> lastmatch=status]; Matches: true]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPathSegments(false) returning [status]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
> -
>>> Using SearchResult: Found: false, Resource: null, Method: null, Error:
>>> null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.FindResourceMethodHandler
>>> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.FindResourceMethodHandler
>>> - Root resource @Path matches exactly so finding root resource method
> in
>>> com.sonatype.benson.rest.api.status.StatusResource
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getQueryParameters(true) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getQueryParameters(true) query string is: null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getQueryParameters(true) encoded query parameters are: []
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getQueryParameters(true) returning []
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> alternateParameter is null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> HttpServletRequest.getHeaders(Accept) returned [application/xml] so
>>> putting into headers cache
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getRequestHeaderInternal(Accept) returning [application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getRequestHeader(Accept) returns [application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> Accept
>>> header is: application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getAcceptHeader() returns application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getAcceptableMediaTypes() returns [application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getRequestHeaderInternal(Content-Type) returning []
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getMediaType() returning null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getAcceptableMediaTypes() returns [application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.FindResourceMethodHandler
>>> - Found root resource method to invoke: MethodMetadata [[consumes=[],
>>> paths=[], produces=[]]formalParameters=[], httpMethod=GET,
> parent=Class:
>>> class com.sonatype.benson.rest.api.status.StatusResource,
>>> reflectionMethod=public javax.ws.rs.core.Response
>>> com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>>
> org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable
>>> parameters list is: []
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list
> to
>>> inject is: null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.InvokeMethodHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler -
>>> Invoking method getStatus of declaring class
>>> com.sonatype.benson.rest.api.status.StatusResource on the instance of a
>>> class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with
>>> parameters []
>>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource -
> Returning
>>> status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) called
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>>> getPath(false) returning status
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getRequestHeaderInternal(Content-Type) returning []
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getMediaType() returning null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
> Finished
>>> request handlers chain and starting response handlers chain: Method:
> GET,
>>> Path: status, MediaType: null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>>
> org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type
>>> from @Produces on method: [application/json, application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>>> getAcceptableMediaTypes() returns [application/xml]
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
> application/xml
>>> to application/json
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
> application/xml
>>> to application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>>> using candidate type application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate
> application/xml
>>> has q value null so adding to possible candidates
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best
> candidate
>>> so using candidate application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type
> will
>>> be set to application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.FlushResultHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>> Response status code set to: 200
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Getting MessageBodyWriter for class type class
>>> com.sonatype.benson.rest.model.Status, genericType class
>>> com.sonatype.benson.rest.model.Status, annotations
> [@javax.ws.rs.GET()],
>>> and media type application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Getting providers by media type by calling
>>> getProvidersByMediaType(application/xml, class
>>> com.sonatype.benson.rest.model.Status)
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get
>>> media type to providers cache for media type application/xml resulted
> in
>>> [OFHolder [genericType=class java.lang.Object,
> mediaType=application/xml,
>>> of=Priority: 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>>> OFHolder [genericType=class java.lang.Object,
> mediaType=application/xml,
>>> of=Priority: 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Found
>>> possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class
>>> java.lang.Object, mediaType=application/xml, of=Priority: 0.100000,
>>> ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>>> OFHolder [genericType=class java.lang.Object,
> mediaType=application/xml,
>>> of=Priority: 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
> of=Priority:
>>> 0.100000, ObjectFactory:
>>>
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Calling
>>>
> org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable(
>>> class com.sonatype.benson.rest.model.Status, class
>>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>>> application/xml )
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Calling
>>>
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>>> class com.sonatype.benson.rest.model.Status, class
>>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>>> application/xml )
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>>
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>>> class com.sonatype.benson.rest.model.Status, class
>>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>>> application/xml ) returned true
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>> Serialization using provider
>>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>>
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0,
>>> class com.sonatype.benson.rest.model.Status, class
>>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>>> application/xml) returned -1
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>>
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0,
>>> class com.sonatype.benson.rest.model.Status, class
>>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>>> application/xml, CaseInsensitiveMultivaluedMap [map=[]],
>>>
> org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85)
>>> being called
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>>> setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap
> [map=[]],
>>> application/xml) entry
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was
>>> applied to the response Content-Type header due to deployment
>>> configuration directive.
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>>> setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Getting ContextResolver for class javax.xml.bind.JAXBContext which has
>>> @Produces compatible with application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> ContextResolvers MediaTypeMap was empty so returning null
>>> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>>> Marshaller obtained [from pool]
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
>>> getCharset(application/xml, null)
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
> getCharset()
>>> returning UTF-8 since requestHeaders was null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> Getting ContextResolver for class
>>> org.apache.wink.common.model.XmlFormattingOptions which has @Produces
>>> compatible with application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>> ContextResolvers MediaTypeMap was empty so returning null
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> Set
>>> response Content-Type to: application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not
>>> find a mapping record so returning original response media type:
>>> application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>> Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
>>> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>>> Marshaller placed back into pool
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>> Flushing headers if not written
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>>> Invoking handler:
>>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>>> Attempting to release resource instance
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
> Releasing
>>> resource instance
>>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
> Finished
>>> response handlers chain
>>> </snip>
>>>
>>> Its overwhelming. What happened to the wire logging? Is that TRACE?
>>>
>>> Personally I think that Wink should consider using DEBUG for
>>> minimal-verbosity describing what is going on, what URI, what params,
>>> which resource was selected... I kinda think that the wire stuff should
> be
>>> in there too, and thats it.
>>>
>>> I don't personally care how mediatypes are compared or selected, ie:
>>>
>>> <snip>
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
> application/xml
>>> to application/json
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
> application/xml
>>> to application/xml
>>> jvm 1 | DEBUG [qtp1721336021-23]
>>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>>> using candidate type application/xml
>>> </snip>
>>>
>>> Perhaps you guys might also want to look at using org.slf4j.Marker's so
>>> that you can distinguish between internal and user logging events. As
> I
>>> know that developers of Wink might think that all of the above is
> DEBUG,
>>> but for a user its really TRACE.
>>>
>>> * * *
>>>
>>> Anyways, those are all minor things, nothing that should hold back a
>>> release... though I'd like to see a 1.1.2 with some of these issues
> fixed
>>> if possible.
>>>
>>> Thanks,
>>>
>>> --jason
>>>
>>>
>>> On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
>>>
>>>> Hi,
>>>>
>>>> I apologize for the delay. Had a few issues to work though.
>>>>
>>>> Now that everything is resolved, I'm calling a vote to release Wink
>>> 1.1.1
>>>> (RC1)
>>>>
>>>> The Maven staging area is at:
>>>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>>>
>>>> The distributions are in:
>>>>
>>>
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>>>
>>>>
>>>>
>>>> This release is tagged at:
>>>>
>>>
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
>>>
>>>>
>>>> (revision 956599)
>>>>
>>>> The vote will be open here for at least 72 hours.
>>>>
>>>> Guide to testing staged releases:
>>>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>>>
>>>> [ ] +1
>>>> [ ] +0
>>>> [ ] -1
>>>>
>>>> -- Jesse Ramos
>>>
>>>
>>>
>>>
>
>
>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jesse A Ramos <jr...@us.ibm.com>.
Thanks, Jason
I went ahead and changed the affected version to 1.1.2 for these issues.
Let me if that's okay or if they should remain as 1.1.1 issues.
-- Jesse Ramos
From:
Jason Dillon <ja...@planet57.com>
To:
wink-dev@incubator.apache.org
Date:
06/22/2010 11:38 AM
Subject:
Re: [VOTE] Release Wink 1.1.1 (RC1)
Sent by:
Jason Dillon <ja...@gmail.com>
Maven fluff:
https://issues.apache.org/jira/browse/WINK-294 (easy to fix)
SLF4J muck:
https://issues.apache.org/jira/browse/WINK-295 (easy to fix)
General logging rant:
https://issues.apache.org/jira/browse/WINK-296 (more complicated,
requires thinking)
--jason
On Jun 22, 2010, at 9:08 AM, Mike Rheinheimer wrote:
> +1, it all looks good to me.
>
> Jason, all good suggestions. I think we can work on these over the
> next few weeks. We'd love to have Jiras and patches though! Hint
> hint. Wink wink. :)
>
> mike
>
>
> On Mon, Jun 21, 2010 at 1:04 PM, Jesse A Ramos <jr...@us.ibm.com>
wrote:
>> Thanks, Jason, for the feedback. I agree that these are items we
should
>> address in a future release. Do you mind opening some JIRAs for these
>> against version 1.1.2?
>>
>> -- Jesse Ramos
>>
>>
>>
>>
>>
>> From:
>> Jason Dillon <ja...@planet57.com>
>> To:
>> wink-dev@incubator.apache.org
>> Date:
>> 06/21/2010 12:48 PM
>> Subject:
>> Re: [VOTE] Release Wink 1.1.1 (RC1)
>> Sent by:
>> Jason Dillon <ja...@gmail.com>
>>
>>
>>
>> There are some missing versions for plugins:
>>
>> <snip>
>> [WARNING] Some problems were encountered while building the effective
>> model for
org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>> must be unique but found duplicate declaration of plugin
>> org.apache.maven.plugins:maven-war-plugin @
>> org.apache.wink:wink:1.1.1-incubating,
>> /Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-common:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-server:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-client:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for
>> org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
>> [WARNING]
'dependencies.dependency.(groupId:artifactId:type:classifier)'
>> must be unique: junit:junit:jar -> duplicate declaration of version (?)
@
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.maven.plugins:maven-dependency-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink:pom:1.1.1-incubating
>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>> must be unique but found duplicate declaration of plugin
>> org.apache.maven.plugins:maven-war-plugin @
>> [WARNING]
>> [WARNING] It is highly recommended to fix these problems because they
>> threaten the stability of your build.
>> [WARNING]
>> [WARNING] For this reason, future Maven versions might no longer
support
>> building such malformed projects.
>> </snip>
>>
>> I also still highly recommend that you upgrade SLF4j to 1.6.0...
>>
>> <snip url="http://slf4j.org/news.html">
>> As of SLF4J version 1.6.0, in the absence of an SLF4J binding,
slf4j-api
>> will default to a no-operation implementation discarding all log
requests.
>> Thus, instead of throwing an exception, SLF4J will emit a single
warning
>> message about the absence of a binding and proceed to discard all log
>> requests without further protest. See also the relevant sectionin the
user
>> manual.
>> </snip>
>>
>> Looks like the logging format is better, though wink still logs way to
>> much at DEBUG IMO. For example, this looks more like TRACE logging to
me:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning jobs
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) encoded path parameters are: [jobs]
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [jobs]
>> </snip>
>>
>> Also the PrivodersRegister logging is still *insane* IMO.
>>
>> I think most of the DEBUG logging that is going on should really be
TRACE
>> logging. For just one simple GET, with DEBUG enabled for
org.apache.wink,
>> this gets spat out:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet -
>> Processing: GET /rest/status (http://localhost:8080/rest/status)
>>
[client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>> Retrieving request processor
>> org.apache.wink.server.internal.RequestProcessor@6128453c using
attribute
>> name org.apache.wink.server.internal.RequestProcessor in servlet
context
>>
ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath is: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath normalized is: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath after context path removed: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath isServlet: true
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath after servlet path removed: /status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath returning requestPath: status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> HttpServletRequest.getHeaders(Content-Type) returned [] so putting into
>> headers cache
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set
>> message context and starting request handlers chain: Method: GET, Path:
>> status, MediaType: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.SearchResultHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.OptionsMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FindRootResourceHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) encoded path parameters are: [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
-
>> Getting URI Info path segments: [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
-
>> Getting stripped path from segments: status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
-
>> Found resource instances: [ResourceRecord: Path: status; ClassMetadata:
>> Class: class com.sonatype.benson.rest.api.status.StatusResource;
>> UriTemplateMatcher: Parent: status; URI: status; Matcher:
>> java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6
>> lastmatch=status]; Matches: true]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler
-
>> Using SearchResult: Found: false, Resource: null, Method: null, Error:
>> null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FindResourceMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.FindResourceMethodHandler
>> - Root resource @Path matches exactly so finding root resource method
in
>> com.sonatype.benson.rest.api.status.StatusResource
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) query string is: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) encoded query parameters are: []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> alternateParameter is null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> HttpServletRequest.getHeaders(Accept) returned [application/xml] so
>> putting into headers cache
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Accept) returning [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeader(Accept) returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
Accept
>> header is: application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptHeader() returns application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.FindResourceMethodHandler
>> - Found root resource method to invoke: MethodMetadata [[consumes=[],
>> paths=[], produces=[]]formalParameters=[], httpMethod=GET,
parent=Class:
>> class com.sonatype.benson.rest.api.status.StatusResource,
>> reflectionMethod=public javax.ws.rs.core.Response
>> com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>>
org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable
>> parameters list is: []
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list
to
>> inject is: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.InvokeMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler -
>> Invoking method getStatus of declaring class
>> com.sonatype.benson.rest.api.status.StatusResource on the instance of a
>> class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with
>> parameters []
>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource -
Returning
>> status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
Finished
>> request handlers chain and starting response handlers chain: Method:
GET,
>> Path: status, MediaType: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>>
org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type
>> from @Produces on method: [application/json, application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
application/xml
>> to application/json
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
application/xml
>> to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>> using candidate type application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate
application/xml
>> has q value null so adding to possible candidates
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best
candidate
>> so using candidate application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type
will
>> be set to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FlushResultHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Response status code set to: 200
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting MessageBodyWriter for class type class
>> com.sonatype.benson.rest.model.Status, genericType class
>> com.sonatype.benson.rest.model.Status, annotations
[@javax.ws.rs.GET()],
>> and media type application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting providers by media type by calling
>> getProvidersByMediaType(application/xml, class
>> com.sonatype.benson.rest.model.Status)
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get
>> media type to providers cache for media type application/xml resulted
in
>> [OFHolder [genericType=class java.lang.Object,
mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object,
mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Found
>> possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class
>> java.lang.Object, mediaType=application/xml, of=Priority: 0.100000,
>> ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object,
mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*,
of=Priority:
>> 0.100000, ObjectFactory:
>>
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Calling
>>
org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml )
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Calling
>>
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml )
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>>
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml ) returned true
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Serialization using provider
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0,
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml) returned -1
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>>
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0,
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml, CaseInsensitiveMultivaluedMap [map=[]],
>>
org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85)
>> being called
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>> setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap
[map=[]],
>> application/xml) entry
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was
>> applied to the response Content-Type header due to deployment
>> configuration directive.
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>> setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting ContextResolver for class javax.xml.bind.JAXBContext which has
>> @Produces compatible with application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> ContextResolvers MediaTypeMap was empty so returning null
>> jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>> Marshaller obtained [from pool]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
>> getCharset(application/xml, null)
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
getCharset()
>> returning UTF-8 since requestHeaders was null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting ContextResolver for class
>> org.apache.wink.common.model.XmlFormattingOptions which has @Produces
>> compatible with application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> ContextResolvers MediaTypeMap was empty so returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
Set
>> response Content-Type to: application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not
>> find a mapping record so returning original response media type:
>> application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
>> jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>> Marshaller placed back into pool
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Flushing headers if not written
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>> Attempting to release resource instance
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
Releasing
>> resource instance
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
Finished
>> response handlers chain
>> </snip>
>>
>> Its overwhelming. What happened to the wire logging? Is that TRACE?
>>
>> Personally I think that Wink should consider using DEBUG for
>> minimal-verbosity describing what is going on, what URI, what params,
>> which resource was selected... I kinda think that the wire stuff should
be
>> in there too, and thats it.
>>
>> I don't personally care how mediatypes are compared or selected, ie:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
application/xml
>> to application/json
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing
application/xml
>> to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>> using candidate type application/xml
>> </snip>
>>
>> Perhaps you guys might also want to look at using org.slf4j.Marker's so
>> that you can distinguish between internal and user logging events. As
I
>> know that developers of Wink might think that all of the above is
DEBUG,
>> but for a user its really TRACE.
>>
>> * * *
>>
>> Anyways, those are all minor things, nothing that should hold back a
>> release... though I'd like to see a 1.1.2 with some of these issues
fixed
>> if possible.
>>
>> Thanks,
>>
>> --jason
>>
>>
>> On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
>>
>>> Hi,
>>>
>>> I apologize for the delay. Had a few issues to work though.
>>>
>>> Now that everything is resolved, I'm calling a vote to release Wink
>> 1.1.1
>>> (RC1)
>>>
>>> The Maven staging area is at:
>>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>>
>>> The distributions are in:
>>>
>>
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>>
>>>
>>>
>>> This release is tagged at:
>>>
>>
https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>>
>>>
>>> (revision 956599)
>>>
>>> The vote will be open here for at least 72 hours.
>>>
>>> Guide to testing staged releases:
>>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>>
>>> [ ] +1
>>> [ ] +0
>>> [ ] -1
>>>
>>> -- Jesse Ramos
>>
>>
>>
>>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jason Dillon <ja...@planet57.com>.
Maven fluff:
https://issues.apache.org/jira/browse/WINK-294 (easy to fix)
SLF4J muck:
https://issues.apache.org/jira/browse/WINK-295 (easy to fix)
General logging rant:
https://issues.apache.org/jira/browse/WINK-296 (more complicated, requires thinking)
--jason
On Jun 22, 2010, at 9:08 AM, Mike Rheinheimer wrote:
> +1, it all looks good to me.
>
> Jason, all good suggestions. I think we can work on these over the
> next few weeks. We'd love to have Jiras and patches though! Hint
> hint. Wink wink. :)
>
> mike
>
>
> On Mon, Jun 21, 2010 at 1:04 PM, Jesse A Ramos <jr...@us.ibm.com> wrote:
>> Thanks, Jason, for the feedback. I agree that these are items we should
>> address in a future release. Do you mind opening some JIRAs for these
>> against version 1.1.2?
>>
>> -- Jesse Ramos
>>
>>
>>
>>
>>
>> From:
>> Jason Dillon <ja...@planet57.com>
>> To:
>> wink-dev@incubator.apache.org
>> Date:
>> 06/21/2010 12:48 PM
>> Subject:
>> Re: [VOTE] Release Wink 1.1.1 (RC1)
>> Sent by:
>> Jason Dillon <ja...@gmail.com>
>>
>>
>>
>> There are some missing versions for plugins:
>>
>> <snip>
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>> must be unique but found duplicate declaration of plugin
>> org.apache.maven.plugins:maven-war-plugin @
>> org.apache.wink:wink:1.1.1-incubating,
>> /Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-common:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-server:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink-client:jar:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.felix:maven-bundle-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for
>> org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
>> [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)'
>> must be unique: junit:junit:jar -> duplicate declaration of version (?) @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
>> [WARNING] 'build.plugins.plugin.version' for
>> org.apache.maven.plugins:maven-dependency-plugin is missing. @
>> [WARNING]
>> [WARNING] Some problems were encountered while building the effective
>> model for org.apache.wink:wink:pom:1.1.1-incubating
>> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
>> must be unique but found duplicate declaration of plugin
>> org.apache.maven.plugins:maven-war-plugin @
>> [WARNING]
>> [WARNING] It is highly recommended to fix these problems because they
>> threaten the stability of your build.
>> [WARNING]
>> [WARNING] For this reason, future Maven versions might no longer support
>> building such malformed projects.
>> </snip>
>>
>> I also still highly recommend that you upgrade SLF4j to 1.6.0...
>>
>> <snip url="http://slf4j.org/news.html">
>> As of SLF4J version 1.6.0, in the absence of an SLF4J binding, slf4j-api
>> will default to a no-operation implementation discarding all log requests.
>> Thus, instead of throwing an exception, SLF4J will emit a single warning
>> message about the absence of a binding and proceed to discard all log
>> requests without further protest. See also the relevant sectionin the user
>> manual.
>> </snip>
>>
>> Looks like the logging format is better, though wink still logs way to
>> much at DEBUG IMO. For example, this looks more like TRACE logging to me:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning jobs
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) encoded path parameters are: [jobs]
>> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [jobs]
>> </snip>
>>
>> Also the PrivodersRegister logging is still *insane* IMO.
>>
>> I think most of the DEBUG logging that is going on should really be TRACE
>> logging. For just one simple GET, with DEBUG enabled for org.apache.wink,
>> this gets spat out:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet -
>> Processing: GET /rest/status (http://localhost:8080/rest/status)
>> [client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>> Retrieving request processor
>> org.apache.wink.server.internal.RequestProcessor@6128453c using attribute
>> name org.apache.wink.server.internal.RequestProcessor in servlet context
>> ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath is: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath normalized is: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath after context path removed: /rest/status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath isServlet: true
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath requestPath after servlet path removed: /status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> buildRequestPath returning requestPath: status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> HttpServletRequest.getHeaders(Content-Type) returned [] so putting into
>> headers cache
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set
>> message context and starting request handlers chain: Method: GET, Path:
>> status, MediaType: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.SearchResultHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.OptionsMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FindRootResourceHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) encoded path parameters are: [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
>> Getting URI Info path segments: [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
>> Getting stripped path from segments: status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
>> Found resource instances: [ResourceRecord: Path: status; ClassMetadata:
>> Class: class com.sonatype.benson.rest.api.status.StatusResource;
>> UriTemplateMatcher: Parent: status; URI: status; Matcher:
>> java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6
>> lastmatch=status]; Matches: true]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPathSegments(false) returning [status]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
>> Using SearchResult: Found: false, Resource: null, Method: null, Error:
>> null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FindResourceMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
>> - Root resource @Path matches exactly so finding root resource method in
>> com.sonatype.benson.rest.api.status.StatusResource
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) query string is: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) encoded query parameters are: []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getQueryParameters(true) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> alternateParameter is null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> HttpServletRequest.getHeaders(Accept) returned [application/xml] so
>> putting into headers cache
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Accept) returning [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeader(Accept) returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - Accept
>> header is: application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptHeader() returns application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
>> - Found root resource method to invoke: MethodMetadata [[consumes=[],
>> paths=[], produces=[]]formalParameters=[], httpMethod=GET, parent=Class:
>> class com.sonatype.benson.rest.api.status.StatusResource,
>> reflectionMethod=public javax.ws.rs.core.Response
>> com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable
>> parameters list is: []
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list to
>> inject is: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.InvokeMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler -
>> Invoking method getStatus of declaring class
>> com.sonatype.benson.rest.api.status.StatusResource on the instance of a
>> class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with
>> parameters []
>> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource - Returning
>> status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) called
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
>> getPath(false) returning status
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getRequestHeaderInternal(Content-Type) returning []
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getMediaType() returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
>> request handlers chain and starting response handlers chain: Method: GET,
>> Path: status, MediaType: null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type
>> from @Produces on method: [application/json, application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
>> getAcceptableMediaTypes() returns [application/xml]
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
>> to application/json
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
>> to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>> using candidate type application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate application/xml
>> has q value null so adding to possible candidates
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best candidate
>> so using candidate application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type will
>> be set to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.FlushResultHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Response status code set to: 200
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting MessageBodyWriter for class type class
>> com.sonatype.benson.rest.model.Status, genericType class
>> com.sonatype.benson.rest.model.Status, annotations [@javax.ws.rs.GET()],
>> and media type application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting providers by media type by calling
>> getProvidersByMediaType(application/xml, class
>> com.sonatype.benson.rest.model.Status)
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get
>> media type to providers cache for media type application/xml resulted in
>> [OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Found
>> possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class
>> java.lang.Object, mediaType=application/xml, of=Priority: 0.100000,
>> ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
>> of=Priority: 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
>> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
>> 0.100000, ObjectFactory:
>> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Calling
>> org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml )
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Calling
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml )
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml ) returned true
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Serialization using provider
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0,
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml) returned -1
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0,
>> class com.sonatype.benson.rest.model.Status, class
>> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
>> application/xml, CaseInsensitiveMultivaluedMap [map=[]],
>> org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85)
>> being called
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>> setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap [map=[]],
>> application/xml) entry
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was
>> applied to the response Content-Type header due to deployment
>> configuration directive.
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
>> setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting ContextResolver for class javax.xml.bind.JAXBContext which has
>> @Produces compatible with application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> ContextResolvers MediaTypeMap was empty so returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>> Marshaller obtained [from pool]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
>> getCharset(application/xml, null)
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils - getCharset()
>> returning UTF-8 since requestHeaders was null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> Getting ContextResolver for class
>> org.apache.wink.common.model.XmlFormattingOptions which has @Produces
>> compatible with application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
>> ContextResolvers MediaTypeMap was empty so returning null
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Set
>> response Content-Type to: application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not
>> find a mapping record so returning original response media type:
>> application/xml
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
>> Marshaller placed back into pool
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
>> Flushing headers if not written
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
>> Invoking handler:
>> org.apache.wink.server.internal.handlers.HeadMethodHandler
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
>> Attempting to release resource instance
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Releasing
>> resource instance
>> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
>> response handlers chain
>> </snip>
>>
>> Its overwhelming. What happened to the wire logging? Is that TRACE?
>>
>> Personally I think that Wink should consider using DEBUG for
>> minimal-verbosity describing what is going on, what URI, what params,
>> which resource was selected... I kinda think that the wire stuff should be
>> in there too, and thats it.
>>
>> I don't personally care how mediatypes are compared or selected, ie:
>>
>> <snip>
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
>> to application/json
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
>> to application/xml
>> jvm 1 | DEBUG [qtp1721336021-23]
>> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
>> using candidate type application/xml
>> </snip>
>>
>> Perhaps you guys might also want to look at using org.slf4j.Marker's so
>> that you can distinguish between internal and user logging events. As I
>> know that developers of Wink might think that all of the above is DEBUG,
>> but for a user its really TRACE.
>>
>> * * *
>>
>> Anyways, those are all minor things, nothing that should hold back a
>> release... though I'd like to see a 1.1.2 with some of these issues fixed
>> if possible.
>>
>> Thanks,
>>
>> --jason
>>
>>
>> On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
>>
>>> Hi,
>>>
>>> I apologize for the delay. Had a few issues to work though.
>>>
>>> Now that everything is resolved, I'm calling a vote to release Wink
>> 1.1.1
>>> (RC1)
>>>
>>> The Maven staging area is at:
>>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>>
>>> The distributions are in:
>>>
>> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>>
>>>
>>>
>>> This release is tagged at:
>>>
>> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>>
>>>
>>> (revision 956599)
>>>
>>> The vote will be open here for at least 72 hours.
>>>
>>> Guide to testing staged releases:
>>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>>
>>> [ ] +1
>>> [ ] +0
>>> [ ] -1
>>>
>>> -- Jesse Ramos
>>
>>
>>
>>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Mike Rheinheimer <ro...@ohmyhead.com>.
+1, it all looks good to me.
Jason, all good suggestions. I think we can work on these over the
next few weeks. We'd love to have Jiras and patches though! Hint
hint. Wink wink. :)
mike
On Mon, Jun 21, 2010 at 1:04 PM, Jesse A Ramos <jr...@us.ibm.com> wrote:
> Thanks, Jason, for the feedback. I agree that these are items we should
> address in a future release. Do you mind opening some JIRAs for these
> against version 1.1.2?
>
> -- Jesse Ramos
>
>
>
>
>
> From:
> Jason Dillon <ja...@planet57.com>
> To:
> wink-dev@incubator.apache.org
> Date:
> 06/21/2010 12:48 PM
> Subject:
> Re: [VOTE] Release Wink 1.1.1 (RC1)
> Sent by:
> Jason Dillon <ja...@gmail.com>
>
>
>
> There are some missing versions for plugins:
>
> <snip>
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
> must be unique but found duplicate declaration of plugin
> org.apache.maven.plugins:maven-war-plugin @
> org.apache.wink:wink:1.1.1-incubating,
> /Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink:wink-common:jar:1.1.1-incubating
> [WARNING] 'build.plugins.plugin.version' for
> org.apache.felix:maven-bundle-plugin is missing. @
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink:wink-server:jar:1.1.1-incubating
> [WARNING] 'build.plugins.plugin.version' for
> org.apache.felix:maven-bundle-plugin is missing. @
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink:wink-client:jar:1.1.1-incubating
> [WARNING] 'build.plugins.plugin.version' for
> org.apache.felix:maven-bundle-plugin is missing. @
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for
> org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
> [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)'
> must be unique: junit:junit:jar -> duplicate declaration of version (?) @
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
> [WARNING] 'build.plugins.plugin.version' for
> org.apache.maven.plugins:maven-dependency-plugin is missing. @
> [WARNING]
> [WARNING] Some problems were encountered while building the effective
> model for org.apache.wink:wink:pom:1.1.1-incubating
> [WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
> must be unique but found duplicate declaration of plugin
> org.apache.maven.plugins:maven-war-plugin @
> [WARNING]
> [WARNING] It is highly recommended to fix these problems because they
> threaten the stability of your build.
> [WARNING]
> [WARNING] For this reason, future Maven versions might no longer support
> building such malformed projects.
> </snip>
>
> I also still highly recommend that you upgrade SLF4j to 1.6.0...
>
> <snip url="http://slf4j.org/news.html">
> As of SLF4J version 1.6.0, in the absence of an SLF4J binding, slf4j-api
> will default to a no-operation implementation discarding all log requests.
> Thus, instead of throwing an exception, SLF4J will emit a single warning
> message about the absence of a binding and proceed to discard all log
> requests without further protest. See also the relevant sectionin the user
> manual.
> </snip>
>
> Looks like the logging format is better, though wink still logs way to
> much at DEBUG IMO. For example, this looks more like TRACE logging to me:
>
> <snip>
> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) called
> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) called
> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) returning jobs
> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) encoded path parameters are: [jobs]
> jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) returning [jobs]
> </snip>
>
> Also the PrivodersRegister logging is still *insane* IMO.
>
> I think most of the DEBUG logging that is going on should really be TRACE
> logging. For just one simple GET, with DEBUG enabled for org.apache.wink,
> this gets spat out:
>
> <snip>
> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet -
> Processing: GET /rest/status (http://localhost:8080/rest/status)
> [client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
> Retrieving request processor
> org.apache.wink.server.internal.RequestProcessor@6128453c using attribute
> name org.apache.wink.server.internal.RequestProcessor in servlet context
> ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath requestPath is: /rest/status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath requestPath normalized is: /rest/status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath after context path removed: /rest/status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath isServlet: true
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath requestPath after servlet path removed: /status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> buildRequestPath returning requestPath: status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) returning status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> HttpServletRequest.getHeaders(Content-Type) returned [] so putting into
> headers cache
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getRequestHeaderInternal(Content-Type) returning []
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getMediaType() returning null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set
> message context and starting request handlers chain: Method: GET, Path:
> status, MediaType: null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.SearchResultHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.OptionsMethodHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.HeadMethodHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.FindRootResourceHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) returning status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) encoded path parameters are: [status]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) returning [status]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
> Getting URI Info path segments: [status]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
> Getting stripped path from segments: status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
> Found resource instances: [ResourceRecord: Path: status; ClassMetadata:
> Class: class com.sonatype.benson.rest.api.status.StatusResource;
> UriTemplateMatcher: Parent: status; URI: status; Matcher:
> java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6
> lastmatch=status]; Matches: true]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPathSegments(false) returning [status]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
> Using SearchResult: Found: false, Resource: null, Method: null, Error:
> null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.FindResourceMethodHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
> - Root resource @Path matches exactly so finding root resource method in
> com.sonatype.benson.rest.api.status.StatusResource
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getQueryParameters(true) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getQueryParameters(true) query string is: null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getQueryParameters(true) encoded query parameters are: []
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getQueryParameters(true) returning []
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> alternateParameter is null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> HttpServletRequest.getHeaders(Accept) returned [application/xml] so
> putting into headers cache
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getRequestHeaderInternal(Accept) returning [application/xml]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getRequestHeader(Accept) returns [application/xml]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - Accept
> header is: application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getAcceptHeader() returns application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getAcceptableMediaTypes() returns [application/xml]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getRequestHeaderInternal(Content-Type) returning []
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getMediaType() returning null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getAcceptableMediaTypes() returns [application/xml]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
> - Found root resource method to invoke: MethodMetadata [[consumes=[],
> paths=[], produces=[]]formalParameters=[], httpMethod=GET, parent=Class:
> class com.sonatype.benson.rest.api.status.StatusResource,
> reflectionMethod=public javax.ws.rs.core.Response
> com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable
> parameters list is: []
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list to
> inject is: null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.InvokeMethodHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler -
> Invoking method getStatus of declaring class
> com.sonatype.benson.rest.api.status.StatusResource on the instance of a
> class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with
> parameters []
> jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource - Returning
> status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) called
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
> getPath(false) returning status
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getRequestHeaderInternal(Content-Type) returning []
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getMediaType() returning null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
> request handlers chain and starting response handlers chain: Method: GET,
> Path: status, MediaType: null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type
> from @Produces on method: [application/json, application/xml]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
> getAcceptableMediaTypes() returns [application/xml]
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
> to application/json
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
> to application/xml
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
> using candidate type application/xml
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate application/xml
> has q value null so adding to possible candidates
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best candidate
> so using candidate application/xml
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type will
> be set to application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.FlushResultHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> Response status code set to: 200
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Getting MessageBodyWriter for class type class
> com.sonatype.benson.rest.model.Status, genericType class
> com.sonatype.benson.rest.model.Status, annotations [@javax.ws.rs.GET()],
> and media type application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Getting providers by media type by calling
> getProvidersByMediaType(application/xml, class
> com.sonatype.benson.rest.model.Status)
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get
> media type to providers cache for media type application/xml resulted in
> [OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
> of=Priority: 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
> OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
> of=Priority: 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Found
> possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class
> java.lang.Object, mediaType=application/xml, of=Priority: 0.100000,
> ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
> OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
> of=Priority: 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
> OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
> 0.100000, ObjectFactory:
> org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Calling
> org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable(
> class com.sonatype.benson.rest.model.Status, class
> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
> application/xml )
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Calling
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
> class com.sonatype.benson.rest.model.Status, class
> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
> application/xml )
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
> class com.sonatype.benson.rest.model.Status, class
> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
> application/xml ) returned true
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> Serialization using provider
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0,
> class com.sonatype.benson.rest.model.Status, class
> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
> application/xml) returned -1
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0,
> class com.sonatype.benson.rest.model.Status, class
> com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
> application/xml, CaseInsensitiveMultivaluedMap [map=[]],
> org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85)
> being called
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
> setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap [map=[]],
> application/xml) entry
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was
> applied to the response Content-Type header due to deployment
> configuration directive.
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
> setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Getting ContextResolver for class javax.xml.bind.JAXBContext which has
> @Produces compatible with application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> ContextResolvers MediaTypeMap was empty so returning null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
> Marshaller obtained [from pool]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
> getCharset(application/xml, null)
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils - getCharset()
> returning UTF-8 since requestHeaders was null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> Getting ContextResolver for class
> org.apache.wink.common.model.XmlFormattingOptions which has @Produces
> compatible with application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
> ContextResolvers MediaTypeMap was empty so returning null
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Set
> response Content-Type to: application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not
> find a mapping record so returning original response media type:
> application/xml
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
> Marshaller placed back into pool
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
> Flushing headers if not written
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
> Invoking handler:
> org.apache.wink.server.internal.handlers.HeadMethodHandler
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
> Attempting to release resource instance
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Releasing
> resource instance
> jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
> response handlers chain
> </snip>
>
> Its overwhelming. What happened to the wire logging? Is that TRACE?
>
> Personally I think that Wink should consider using DEBUG for
> minimal-verbosity describing what is going on, what URI, what params,
> which resource was selected... I kinda think that the wire stuff should be
> in there too, and thats it.
>
> I don't personally care how mediatypes are compared or selected, ie:
>
> <snip>
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
> to application/json
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
> to application/xml
> jvm 1 | DEBUG [qtp1721336021-23]
> o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
> using candidate type application/xml
> </snip>
>
> Perhaps you guys might also want to look at using org.slf4j.Marker's so
> that you can distinguish between internal and user logging events. As I
> know that developers of Wink might think that all of the above is DEBUG,
> but for a user its really TRACE.
>
> * * *
>
> Anyways, those are all minor things, nothing that should hold back a
> release... though I'd like to see a 1.1.2 with some of these issues fixed
> if possible.
>
> Thanks,
>
> --jason
>
>
> On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
>
>> Hi,
>>
>> I apologize for the delay. Had a few issues to work though.
>>
>> Now that everything is resolved, I'm calling a vote to release Wink
> 1.1.1
>> (RC1)
>>
>> The Maven staging area is at:
>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>
>> The distributions are in:
>>
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>>
>>
>> This release is tagged at:
>>
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
>>
>> (revision 956599)
>>
>> The vote will be open here for at least 72 hours.
>>
>> Guide to testing staged releases:
>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>
>> [ ] +1
>> [ ] +0
>> [ ] -1
>>
>> -- Jesse Ramos
>
>
>
>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jesse A Ramos <jr...@us.ibm.com>.
Thanks, Jason, for the feedback. I agree that these are items we should
address in a future release. Do you mind opening some JIRAs for these
against version 1.1.2?
-- Jesse Ramos
From:
Jason Dillon <ja...@planet57.com>
To:
wink-dev@incubator.apache.org
Date:
06/21/2010 12:48 PM
Subject:
Re: [VOTE] Release Wink 1.1.1 (RC1)
Sent by:
Jason Dillon <ja...@gmail.com>
There are some missing versions for plugins:
<snip>
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
must be unique but found duplicate declaration of plugin
org.apache.maven.plugins:maven-war-plugin @
org.apache.wink:wink:1.1.1-incubating,
/Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink:wink-common:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for
org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink:wink-server:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for
org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink:wink-client:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for
org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for
org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
[WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)'
must be unique: junit:junit:jar -> duplicate declaration of version (?) @
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for
org.apache.maven.plugins:maven-dependency-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective
model for org.apache.wink:wink:pom:1.1.1-incubating
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)'
must be unique but found duplicate declaration of plugin
org.apache.maven.plugins:maven-war-plugin @
[WARNING]
[WARNING] It is highly recommended to fix these problems because they
threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support
building such malformed projects.
</snip>
I also still highly recommend that you upgrade SLF4j to 1.6.0...
<snip url="http://slf4j.org/news.html">
As of SLF4J version 1.6.0, in the absence of an SLF4J binding, slf4j-api
will default to a no-operation implementation discarding all log requests.
Thus, instead of throwing an exception, SLF4J will emit a single warning
message about the absence of a binding and proceed to discard all log
requests without further protest. See also the relevant sectionin the user
manual.
</snip>
Looks like the logging format is better, though wink still logs way to
much at DEBUG IMO. For example, this looks more like TRACE logging to me:
<snip>
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
getPath(false) called
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
getPath(false) returning jobs
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) encoded path parameters are: [jobs]
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) returning [jobs]
</snip>
Also the PrivodersRegister logging is still *insane* IMO.
I think most of the DEBUG logging that is going on should really be TRACE
logging. For just one simple GET, with DEBUG enabled for org.apache.wink,
this gets spat out:
<snip>
jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet -
Processing: GET /rest/status (http://localhost:8080/rest/status)
[client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
Retrieving request processor
org.apache.wink.server.internal.RequestProcessor@6128453c using attribute
name org.apache.wink.server.internal.RequestProcessor in servlet context
ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath requestPath is: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath requestPath normalized is: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath after context path removed: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath isServlet: true
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath requestPath after servlet path removed: /status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
buildRequestPath returning requestPath: status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
HttpServletRequest.getHeaders(Content-Type) returned [] so putting into
headers cache
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set
message context and starting request handlers chain: Method: GET, Path:
status, MediaType: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.SearchResultHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.OptionsMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.HeadMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.FindRootResourceHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) encoded path parameters are: [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) returning [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
Getting URI Info path segments: [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
Getting stripped path from segments: status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
Found resource instances: [ResourceRecord: Path: status; ClassMetadata:
Class: class com.sonatype.benson.rest.api.status.StatusResource;
UriTemplateMatcher: Parent: status; URI: status; Matcher:
java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6
lastmatch=status]; Matches: true]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPathSegments(false) returning [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler -
Using SearchResult: Found: false, Resource: null, Method: null, Error:
null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.FindResourceMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
- Root resource @Path matches exactly so finding root resource method in
com.sonatype.benson.rest.api.status.StatusResource
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getQueryParameters(true) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getQueryParameters(true) query string is: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getQueryParameters(true) encoded query parameters are: []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getQueryParameters(true) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
alternateParameter is null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
HttpServletRequest.getHeaders(Accept) returned [application/xml] so
putting into headers cache
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getRequestHeaderInternal(Accept) returning [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getRequestHeader(Accept) returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - Accept
header is: application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getAcceptHeader() returns application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler
- Found root resource method to invoke: MethodMetadata [[consumes=[],
paths=[], produces=[]]formalParameters=[], httpMethod=GET, parent=Class:
class com.sonatype.benson.rest.api.status.StatusResource,
reflectionMethod=public javax.ws.rs.core.Response
com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable
parameters list is: []
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list to
inject is: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.InvokeMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler -
Invoking method getStatus of declaring class
com.sonatype.benson.rest.api.status.StatusResource on the instance of a
class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with
parameters []
jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource - Returning
status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl -
getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
request handlers chain and starting response handlers chain: Method: GET,
Path: status, MediaType: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type
from @Produces on method: [application/json, application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl -
getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
to application/json
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
to application/xml
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
using candidate type application/xml
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate application/xml
has q value null so adding to possible candidates
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best candidate
so using candidate application/xml
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type will
be set to application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.FlushResultHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
Response status code set to: 200
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Getting MessageBodyWriter for class type class
com.sonatype.benson.rest.model.Status, genericType class
com.sonatype.benson.rest.model.Status, annotations [@javax.ws.rs.GET()],
and media type application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Getting providers by media type by calling
getProvidersByMediaType(application/xml, class
com.sonatype.benson.rest.model.Status)
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get
media type to providers cache for media type application/xml resulted in
[OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
of=Priority: 0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
of=Priority: 0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Found
possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class
java.lang.Object, mediaType=application/xml, of=Priority: 0.100000,
ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
OFHolder [genericType=class java.lang.Object, mediaType=application/xml,
of=Priority: 0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db],
OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority:
0.100000, ObjectFactory:
org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Calling
org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable(
class com.sonatype.benson.rest.model.Status, class
com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
application/xml )
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Calling
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
class com.sonatype.benson.rest.model.Status, class
com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
application/xml )
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable(
class com.sonatype.benson.rest.model.Status, class
com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
application/xml ) returned true
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
Serialization using provider
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0,
class com.sonatype.benson.rest.model.Status, class
com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
application/xml) returned -1
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0,
class com.sonatype.benson.rest.model.Status, class
com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()],
application/xml, CaseInsensitiveMultivaluedMap [map=[]],
org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85)
being called
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap [map=[]],
application/xml) entry
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was
applied to the response Content-Type header due to deployment
configuration directive.
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster -
setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Getting ContextResolver for class javax.xml.bind.JAXBContext which has
@Produces compatible with application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
ContextResolvers MediaTypeMap was empty so returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
Marshaller obtained [from pool]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils -
getCharset(application/xml, null)
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils - getCharset()
returning UTF-8 since requestHeaders was null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
Getting ContextResolver for class
org.apache.wink.common.model.XmlFormattingOptions which has @Produces
compatible with application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry -
ContextResolvers MediaTypeMap was empty so returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Set
response Content-Type to: application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not
find a mapping record so returning original response media type:
application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider -
Marshaller placed back into pool
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler -
Flushing headers if not written
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain -
Invoking handler:
org.apache.wink.server.internal.handlers.HeadMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor -
Attempting to release resource instance
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Releasing
resource instance
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished
response handlers chain
</snip>
Its overwhelming. What happened to the wire logging? Is that TRACE?
Personally I think that Wink should consider using DEBUG for
minimal-verbosity describing what is going on, what URI, what params,
which resource was selected... I kinda think that the wire stuff should be
in there too, and thats it.
I don't personally care how mediatypes are compared or selected, ie:
<snip>
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
to application/json
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml
to application/xml
jvm 1 | DEBUG [qtp1721336021-23]
o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so
using candidate type application/xml
</snip>
Perhaps you guys might also want to look at using org.slf4j.Marker's so
that you can distinguish between internal and user logging events. As I
know that developers of Wink might think that all of the above is DEBUG,
but for a user its really TRACE.
* * *
Anyways, those are all minor things, nothing that should hold back a
release... though I'd like to see a 1.1.2 with some of these issues fixed
if possible.
Thanks,
--jason
On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
> Hi,
>
> I apologize for the delay. Had a few issues to work though.
>
> Now that everything is resolved, I'm calling a vote to release Wink
1.1.1
> (RC1)
>
> The Maven staging area is at:
> https://repository.apache.org/content/repositories/orgapachewink-003/
>
> The distributions are in:
>
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>
> This release is tagged at:
>
https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
> (revision 956599)
>
> The vote will be open here for at least 72 hours.
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> [ ] +1
> [ ] +0
> [ ] -1
>
> -- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jason Dillon <ja...@planet57.com>.
There are some missing versions for plugins:
<snip>
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink-component-test-support:jar:1.1.1-incubating
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.apache.maven.plugins:maven-war-plugin @ org.apache.wink:wink:1.1.1-incubating, /Users/jason/ws/tmp/wink-1.1.1-incubating/pom.xml
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink-common:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink-server:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink-client:jar:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for org.apache.felix:maven-bundle-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink-client-apache-httpclient:jar:1.1.1-incubating
[WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: junit:junit:jar -> duplicate declaration of version (?) @
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink.example:WebDAV:war:1.1.1-incubating
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-dependency-plugin is missing. @
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.wink:wink:pom:1.1.1-incubating
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.apache.maven.plugins:maven-war-plugin @
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
</snip>
I also still highly recommend that you upgrade SLF4j to 1.6.0...
<snip url="http://slf4j.org/news.html">
As of SLF4J version 1.6.0, in the absence of an SLF4J binding, slf4j-api will default to a no-operation implementation discarding all log requests. Thus, instead of throwing an exception, SLF4J will emit a single warning message about the absence of a binding and proceed to discard all log requests without further protest. See also the relevant sectionin the user manual.
</snip>
Looks like the logging format is better, though wink still logs way to much at DEBUG IMO. For example, this looks more like TRACE logging to me:
<snip>
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl - getPath(false) called
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl - getPath(false) returning jobs
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) encoded path parameters are: [jobs]
jvm 1 | DEBUG [qtp1721336021-24] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) returning [jobs]
</snip>
Also the PrivodersRegister logging is still *insane* IMO.
I think most of the DEBUG logging that is going on should really be TRACE logging. For just one simple GET, with DEBUG enabled for org.apache.wink, this gets spat out:
<snip>
jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.s.i.w.RestServlet - Processing: GET /rest/status (http://localhost:8080/rest/status) [client=0.1-SNAPSHOT;model=0.1-SNAPSHOT;id=a74c4a57-97d7-4ef6-9676-469db4b09d75]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Retrieving request processor org.apache.wink.server.internal.RequestProcessor@6128453c using attribute name org.apache.wink.server.internal.RequestProcessor in servlet context ServletContext@6063f5af{/,file:/Users/jason/ws/sonatype/benson/target/benson-0.1-SNAPSHOT/runtime/apps/benson/webapps/benson/}
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath requestPath is: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath requestPath normalized is: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath after context path removed: /rest/status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath isServlet: true
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath requestPath after servlet path removed: /status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - buildRequestPath returning requestPath: status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - HttpServletRequest.getHeaders(Content-Type) returned [] so putting into headers cache
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Set message context and starting request handlers chain: Method: GET, Path: status, MediaType: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.SearchResultHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.OptionsMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.HeadMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.FindRootResourceHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) encoded path parameters are: [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) returning [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler - Getting URI Info path segments: [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler - Getting stripped path from segments: status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler - Found resource instances: [ResourceRecord: Path: status; ClassMetadata: Class: class com.sonatype.benson.rest.api.status.StatusResource; UriTemplateMatcher: Parent: status; URI: status; Matcher: java.util.regex.Matcher[pattern=(\Qstatus\E)((?:/.*)?) region=0,6 lastmatch=status]; Matches: true]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPathSegments(false) returning [status]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindRootResourceHandler - Using SearchResult: Found: false, Resource: null, Method: null, Error: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.FindResourceMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler - Root resource @Path matches exactly so finding root resource method in com.sonatype.benson.rest.api.status.StatusResource
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getQueryParameters(true) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getQueryParameters(true) query string is: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getQueryParameters(true) encoded query parameters are: []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getQueryParameters(true) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - alternateParameter is null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - HttpServletRequest.getHeaders(Accept) returned [application/xml] so putting into headers cache
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getRequestHeaderInternal(Accept) returning [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getRequestHeader(Accept) returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - Accept header is: application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getAcceptHeader() returns application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FindResourceMethodHandler - Found root resource method to invoke: MethodMetadata [[consumes=[], paths=[], produces=[]]formalParameters=[], httpMethod=GET, parent=Class: class com.sonatype.benson.rest.api.status.StatusResource, reflectionMethod=public javax.ws.rs.core.Response com.sonatype.benson.rest.api.status.StatusResource.getStatus()]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.CreateInvocationParametersHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.CreateInvocationParametersHandler - Formal Injectable parameters list is: []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.CreateInvocationParametersHandler - Actual parameters list to inject is: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.InvokeMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.InvokeMethodHandler - Invoking method getStatus of declaring class com.sonatype.benson.rest.api.status.StatusResource on the instance of a class com.sonatype.benson.rest.api.status.StatusResource@ab14733 with parameters []
jvm 1 | DEBUG [qtp1721336021-23] c.s.b.r.a.s.StatusResource - Returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.UriInfoImpl - getPath(false) returning status
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getRequestHeaderInternal(Content-Type) returning []
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getMediaType() returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished request handlers chain and starting response handlers chain: Method: GET, Path: status, MediaType: null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.PopulateResponseStatusHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.PopulateResponseMediaTypeHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Determining Content-Type from @Produces on method: [application/json, application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.HttpHeadersImpl - getAcceptableMediaTypes() returns [application/xml]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml to application/json
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml to application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so using candidate type application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Candidate application/xml has q value null so adding to possible candidates
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - No previous best candidate so using candidate application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Response Content-Type will be set to application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.FlushResultHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Response status code set to: 200
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Getting MessageBodyWriter for class type class com.sonatype.benson.rest.model.Status, genericType class com.sonatype.benson.rest.model.Status, annotations [@javax.ws.rs.GET()], and media type application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Getting providers by media type by calling getProvidersByMediaType(application/xml, class com.sonatype.benson.rest.model.Status)
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Get media type to providers cache for media type application/xml resulted in [OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db], OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Found possible MessageBodyWriter ObjectFactories [OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db], OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@4df2a9da], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@561140db], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.GuiceInjectorLifeCycleManager$GuiceSingletonObjectFactory@531a700a]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Calling org.apache.wink.common.internal.providers.entity.xml.JAXBCollectionXmlProvider@9a303d3.isWriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()], application/xml )
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Calling org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()], application/xml )
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.isWriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()], application/xml ) returned true
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Serialization using provider org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.getSize(com.sonatype.benson.rest.model.Status@7cb96ac0, class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()], application/xml) returned -1
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - org.apache.wink.common.internal.providers.entity.xml.JAXBXmlProvider@14dd18cc.writeTo(com.sonatype.benson.rest.model.Status@7cb96ac0, class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [@javax.ws.rs.GET()], application/xml, CaseInsensitiveMultivaluedMap [map=[]], org.apache.wink.server.internal.handlers.FlushResultHandler$FlushHeadersOutputStream@39a4eb85) being called
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - setDefaultCharsetOnMediaTypeHeader(CaseInsensitiveMultivaluedMap [map=[]], application/xml) entry
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - No default charset was applied to the response Content-Type header due to deployment configuration directive.
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.c.ServerMediaTypeCharsetAdjuster - setDefaultCharsetOnMediaTypeHeader() exit returning application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Getting ContextResolver for class javax.xml.bind.JAXBContext which has @Produces compatible with application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - ContextResolvers MediaTypeMap was empty so returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider - Marshaller obtained [from pool]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils - getCharset(application/xml, null)
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.u.ProviderUtils - getCharset() returning UTF-8 since requestHeaders was null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Getting ContextResolver for class org.apache.wink.common.model.XmlFormattingOptions which has @Produces compatible with application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - ContextResolvers MediaTypeMap was empty so returning null
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Set response Content-Type to: application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.MediaTypeMapper - Did not find a mapping record so returning original response media type: application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Flushing headers: CaseInsensitiveMultivaluedMap [map=[]]
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.p.e.x.AbstractJAXBProvider - Marshaller placed back into pool
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - Flushing headers if not written
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.h.AbstractHandlersChain - Invoking handler: org.apache.wink.server.internal.handlers.HeadMethodHandler
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Attempting to release resource instance
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Releasing resource instance
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.RequestProcessor - Finished response handlers chain
</snip>
Its overwhelming. What happened to the wire logging? Is that TRACE?
Personally I think that Wink should consider using DEBUG for minimal-verbosity describing what is going on, what URI, what params, which resource was selected... I kinda think that the wire stuff should be in there too, and thats it.
I don't personally care how mediatypes are compared or selected, ie:
<snip>
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml to application/json
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - Comparing application/xml to application/xml
jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.PopulateResponseMediaTypeHandler - MediaType compatible so using candidate type application/xml
</snip>
Perhaps you guys might also want to look at using org.slf4j.Marker's so that you can distinguish between internal and user logging events. As I know that developers of Wink might think that all of the above is DEBUG, but for a user its really TRACE.
* * *
Anyways, those are all minor things, nothing that should hold back a release... though I'd like to see a 1.1.2 with some of these issues fixed if possible.
Thanks,
--jason
On Jun 21, 2010, at 7:39 AM, Jesse A Ramos wrote:
> Hi,
>
> I apologize for the delay. Had a few issues to work though.
>
> Now that everything is resolved, I'm calling a vote to release Wink 1.1.1
> (RC1)
>
> The Maven staging area is at:
> https://repository.apache.org/content/repositories/orgapachewink-003/
>
> The distributions are in:
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>
> This release is tagged at:
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
> (revision 956599)
>
> The vote will be open here for at least 72 hours.
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> [ ] +1
> [ ] +0
> [ ] -1
>
> -- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Nicholas L Gallardo <nl...@us.ibm.com>.
+1
Pulled the tag and everything looks clean to me.
Jesse A
Ramos/Austin/IBM@
IBMUS To
"wink-dev@incubator.apache.org"
06/21/2010 09:39 <wi...@incubator.apache.org>
AM cc
Subject
Please respond to [VOTE] Release Wink 1.1.1 (RC1)
wink-dev@incubato
r.apache.org
Hi,
I apologize for the delay. Had a few issues to work though.
Now that everything is resolved, I'm calling a vote to release Wink 1.1.1
(RC1)
The Maven staging area is at:
https://repository.apache.org/content/repositories/orgapachewink-003/
The distributions are in:
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
This release is tagged at:
https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
(revision 956599)
The vote will be open here for at least 72 hours.
Guide to testing staged releases:
http://maven.apache.org/guides/development/guide-testing-releases.html
[ ] +1
[ ] +0
[ ] -1
-- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Bryant Luk <br...@gmail.com>.
I made the changes to the LICENSE headers and changed the
maven-license-plugin in the root pom.xml to not exclude pom.xmls. All
files listed on the exclusion list (to my knowledge) either a) really
do have the LICENSE header but don't have a file extension so the
Maven plugin can't do the check (since it can't determine if it's a
plain text file comment needed or an XML comment needed), b) are
binary files, or c) cannot have the LICENSE header.
I think this solves the LICENSE header issues Kevan ran into. If
anyone encounters an error building, please send an e-mail.
On Fri, Jun 25, 2010 at 9:50 AM, Bryant Luk <br...@gmail.com> wrote:
> I'll fix up some of the warnings as well. Need a few.
>
> On Fri, Jun 25, 2010 at 8:14 AM, Jesse A Ramos <jr...@us.ibm.com> wrote:
>> Thanks, Kevin, for pointing out these issues. I'll look into them today.
>>
>> -- Jesse
>>
>>
>>
>>
>>
>> From:
>> Kevan Miller <ke...@gmail.com>
>> To:
>> wink-dev@incubator.apache.org
>> Date:
>> 06/24/2010 10:26 PM
>> Subject:
>> Re: [VOTE] Release Wink 1.1.1 (RC1)
>>
>>
>>
>> A couple of problems.
>>
>> First, wink-itests/wink-itest/wink-itest-contentencode/pom.xml needs an
>> apache source license header. There are some number of additional files
>> without a license header. They appear to be trivial, machine generated, or
>> syntactically problematic to include a license header.
>>
>> Next, there's something wrong with the checksums. I spot checked several
>> files. For example
>> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/apache-wink-1.1.1-incubating-src.tar.gz
>>
>>
>> The signature looks ok, but all of the checksums were bad (same for
>> apache-wink-1.1.1-incubating.zip...
>>
>> 2e6620ec280aa808915cc1f7c7bb692e126ba72b
>> apache-wink-1.1.1-incubating-src.tar.gz.asc
>> 949be5a28efc55eab93d3d79de49874b00667415
>>
>> 8c09087c954638444245a8127c3dc3fa
>> apache-wink-1.1.1-incubating-src.tar.gz.asc
>> 590752ae0e311f4230f315f5111b448a
>>
>> 66100abe60f2dcda9dfc246f38190a3f63d9e5eb
>> apache-wink-1.1.1-incubating-src.tar.gz
>> 6a2609297db89addf13b1ae8e53830644479766b
>>
>> 6d224c05dc417b5c0aa47896cfc860fb apache-wink-1.1.1-incubating-src.tar.gz
>> d2d27d70eb6e3565e1f6d9ed95b04812
>>
>> These problems need to be fixed. Here's my -1
>>
>> --kevan
>>
>>
>> On Jun 21, 2010, at 10:39 AM, Jesse A Ramos wrote:
>>
>>> Hi,
>>>
>>> I apologize for the delay. Had a few issues to work though.
>>>
>>> Now that everything is resolved, I'm calling a vote to release Wink
>> 1.1.1
>>> (RC1)
>>>
>>> The Maven staging area is at:
>>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>>
>>> The distributions are in:
>>>
>> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>>
>>>
>>>
>>> This release is tagged at:
>>>
>> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>>
>>>
>>> (revision 956599)
>>>
>>> The vote will be open here for at least 72 hours.
>>>
>>> Guide to testing staged releases:
>>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>>
>>> [ ] +1
>>> [ ] +0
>>> [ ] -1
>>>
>>> -- Jesse Ramos
>>
>>
>>
>>
>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Bryant Luk <br...@gmail.com>.
I'll fix up some of the warnings as well. Need a few.
On Fri, Jun 25, 2010 at 8:14 AM, Jesse A Ramos <jr...@us.ibm.com> wrote:
> Thanks, Kevin, for pointing out these issues. I'll look into them today.
>
> -- Jesse
>
>
>
>
>
> From:
> Kevan Miller <ke...@gmail.com>
> To:
> wink-dev@incubator.apache.org
> Date:
> 06/24/2010 10:26 PM
> Subject:
> Re: [VOTE] Release Wink 1.1.1 (RC1)
>
>
>
> A couple of problems.
>
> First, wink-itests/wink-itest/wink-itest-contentencode/pom.xml needs an
> apache source license header. There are some number of additional files
> without a license header. They appear to be trivial, machine generated, or
> syntactically problematic to include a license header.
>
> Next, there's something wrong with the checksums. I spot checked several
> files. For example
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/apache-wink-1.1.1-incubating-src.tar.gz
>
>
> The signature looks ok, but all of the checksums were bad (same for
> apache-wink-1.1.1-incubating.zip...
>
> 2e6620ec280aa808915cc1f7c7bb692e126ba72b
> apache-wink-1.1.1-incubating-src.tar.gz.asc
> 949be5a28efc55eab93d3d79de49874b00667415
>
> 8c09087c954638444245a8127c3dc3fa
> apache-wink-1.1.1-incubating-src.tar.gz.asc
> 590752ae0e311f4230f315f5111b448a
>
> 66100abe60f2dcda9dfc246f38190a3f63d9e5eb
> apache-wink-1.1.1-incubating-src.tar.gz
> 6a2609297db89addf13b1ae8e53830644479766b
>
> 6d224c05dc417b5c0aa47896cfc860fb apache-wink-1.1.1-incubating-src.tar.gz
> d2d27d70eb6e3565e1f6d9ed95b04812
>
> These problems need to be fixed. Here's my -1
>
> --kevan
>
>
> On Jun 21, 2010, at 10:39 AM, Jesse A Ramos wrote:
>
>> Hi,
>>
>> I apologize for the delay. Had a few issues to work though.
>>
>> Now that everything is resolved, I'm calling a vote to release Wink
> 1.1.1
>> (RC1)
>>
>> The Maven staging area is at:
>> https://repository.apache.org/content/repositories/orgapachewink-003/
>>
>> The distributions are in:
>>
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>>
>>
>> This release is tagged at:
>>
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
>>
>> (revision 956599)
>>
>> The vote will be open here for at least 72 hours.
>>
>> Guide to testing staged releases:
>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>
>> [ ] +1
>> [ ] +0
>> [ ] -1
>>
>> -- Jesse Ramos
>
>
>
>
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Jesse A Ramos <jr...@us.ibm.com>.
Thanks, Kevin, for pointing out these issues. I'll look into them today.
-- Jesse
From:
Kevan Miller <ke...@gmail.com>
To:
wink-dev@incubator.apache.org
Date:
06/24/2010 10:26 PM
Subject:
Re: [VOTE] Release Wink 1.1.1 (RC1)
A couple of problems.
First, wink-itests/wink-itest/wink-itest-contentencode/pom.xml needs an
apache source license header. There are some number of additional files
without a license header. They appear to be trivial, machine generated, or
syntactically problematic to include a license header.
Next, there's something wrong with the checksums. I spot checked several
files. For example
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/apache-wink-1.1.1-incubating-src.tar.gz
The signature looks ok, but all of the checksums were bad (same for
apache-wink-1.1.1-incubating.zip...
2e6620ec280aa808915cc1f7c7bb692e126ba72b
apache-wink-1.1.1-incubating-src.tar.gz.asc
949be5a28efc55eab93d3d79de49874b00667415
8c09087c954638444245a8127c3dc3fa
apache-wink-1.1.1-incubating-src.tar.gz.asc
590752ae0e311f4230f315f5111b448a
66100abe60f2dcda9dfc246f38190a3f63d9e5eb
apache-wink-1.1.1-incubating-src.tar.gz
6a2609297db89addf13b1ae8e53830644479766b
6d224c05dc417b5c0aa47896cfc860fb apache-wink-1.1.1-incubating-src.tar.gz
d2d27d70eb6e3565e1f6d9ed95b04812
These problems need to be fixed. Here's my -1
--kevan
On Jun 21, 2010, at 10:39 AM, Jesse A Ramos wrote:
> Hi,
>
> I apologize for the delay. Had a few issues to work though.
>
> Now that everything is resolved, I'm calling a vote to release Wink
1.1.1
> (RC1)
>
> The Maven staging area is at:
> https://repository.apache.org/content/repositories/orgapachewink-003/
>
> The distributions are in:
>
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>
> This release is tagged at:
>
https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
> (revision 956599)
>
> The vote will be open here for at least 72 hours.
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> [ ] +1
> [ ] +0
> [ ] -1
>
> -- Jesse Ramos
Re: [VOTE] Release Wink 1.1.1 (RC1)
Posted by Kevan Miller <ke...@gmail.com>.
A couple of problems.
First, wink-itests/wink-itest/wink-itest-contentencode/pom.xml needs an apache source license header. There are some number of additional files without a license header. They appear to be trivial, machine generated, or syntactically problematic to include a license header.
Next, there's something wrong with the checksums. I spot checked several files. For example
https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/apache-wink-1.1.1-incubating-src.tar.gz
The signature looks ok, but all of the checksums were bad (same for apache-wink-1.1.1-incubating.zip...
2e6620ec280aa808915cc1f7c7bb692e126ba72b apache-wink-1.1.1-incubating-src.tar.gz.asc
949be5a28efc55eab93d3d79de49874b00667415
8c09087c954638444245a8127c3dc3fa apache-wink-1.1.1-incubating-src.tar.gz.asc
590752ae0e311f4230f315f5111b448a
66100abe60f2dcda9dfc246f38190a3f63d9e5eb apache-wink-1.1.1-incubating-src.tar.gz
6a2609297db89addf13b1ae8e53830644479766b
6d224c05dc417b5c0aa47896cfc860fb apache-wink-1.1.1-incubating-src.tar.gz
d2d27d70eb6e3565e1f6d9ed95b04812
These problems need to be fixed. Here's my -1
--kevan
On Jun 21, 2010, at 10:39 AM, Jesse A Ramos wrote:
> Hi,
>
> I apologize for the delay. Had a few issues to work though.
>
> Now that everything is resolved, I'm calling a vote to release Wink 1.1.1
> (RC1)
>
> The Maven staging area is at:
> https://repository.apache.org/content/repositories/orgapachewink-003/
>
> The distributions are in:
> https://repository.apache.org/content/repositories/orgapachewink-003/org/apache/wink/apache-wink/1.1.1-incubating/
>
>
> This release is tagged at:
> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.1.1-incubating/
>
> (revision 956599)
>
> The vote will be open here for at least 72 hours.
>
> Guide to testing staged releases:
> http://maven.apache.org/guides/development/guide-testing-releases.html
>
> [ ] +1
> [ ] +0
> [ ] -1
>
> -- Jesse Ramos