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 <[email protected]> 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 <[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
>>
>>
>>
>>
