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 <[email protected]> To: [email protected] Date: 06/21/2010 12:48 PM Subject: Re: [VOTE] Release Wink 1.1.1 (RC1) Sent by: Jason Dillon <[email protected]> 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.requestproces...@6128453c using attribute name org.apache.wink.server.internal.RequestProcessor in servlet context servletcont...@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.statusresou...@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 [[email protected]()], 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$guicesingletonobjectfact...@561140db], OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@531a700a], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@4df2a9da], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@561140db], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@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$guicesingletonobjectfact...@561140db], OFHolder [genericType=class java.lang.Object, mediaType=application/xml, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@531a700a], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@4df2a9da], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@561140db], OFHolder [genericType=class java.lang.Object, mediaType=*/*, of=Priority: 0.100000, ObjectFactory: org.apache.wink.guice.server.internal.lifecycle.guiceinjectorlifecyclemanager$guicesingletonobjectfact...@531a700a]] jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Calling org.apache.wink.common.internal.providers.entity.xml.jaxbcollectionxmlprovi...@9a303d3.iswriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [[email protected]()], application/xml ) jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - Calling org.apache.wink.common.internal.providers.entity.xml.jaxbxmlprovi...@14dd18cc.iswriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [[email protected]()], application/xml ) jvm 1 | DEBUG [qtp1721336021-23] o.a.w.c.i.r.ProvidersRegistry - org.apache.wink.common.internal.providers.entity.xml.jaxbxmlprovi...@14dd18cc.iswriteable( class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [[email protected]()], 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.jaxbxmlprovi...@14dd18cc.getsize(com.sonatype.benson.rest.model.sta...@7cb96ac0, class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [[email protected]()], application/xml) returned -1 jvm 1 | DEBUG [qtp1721336021-23] o.a.w.s.i.h.FlushResultHandler - org.apache.wink.common.internal.providers.entity.xml.jaxbxmlprovi...@14dd18cc.writeto(com.sonatype.benson.rest.model.sta...@7cb96ac0, class com.sonatype.benson.rest.model.Status, class com.sonatype.benson.rest.model.Status, [[email protected]()], application/xml, CaseInsensitiveMultivaluedMap [map=[]], org.apache.wink.server.internal.handlers.flushresulthandler$flushheadersoutputstr...@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
