I see the point now. Many thanks for the detailed explanation. (I am sorry I had missed out reading release_notes before).
So are we having a new release candidate to take care of issues 1) and 3) mentioned before? Thanks, Shiva Kumar H R On Sun, Oct 18, 2009 at 10:32 AM, Bryant Luk <[email protected]> wrote: > The slf4j-simple JAR that's included with the binary distribution logs > any INFO message (or higher like WARNING) to System.err (and cannot be > configured differently I believe). It is the simplest logging > framework SLF4J JAR that "works". > > It is expected that users will replace the slf4j-simple JAR with their > preferred logging SLF4J bridging JAR as mentioned in the release > notes. So for some environments, that's the slf4j-jdk14 JAR, others > it is slf4j-log4j, etc. Depending on their environment, out of the > box, they probably won't get any INFO messages to console but maybe to > a log file. For instance, in Geronimo, you shouldn't even package > slf4j-api or slf4j-simple JARs in your application since the > appropriate SLF4J JARs are already included in the classpath and > whatever configuration Geronimo has, you will be using (so logs would > go to a file). > > By using slf4j in general, we can adopt whatever environment logging > mechanism is used avoiding a few issues. While slf4j-simple isn't at > all ideal, it's meant merely as a placeholder so that users can choose > whatever they want. > > Also, my current thinking is that it alerts the user up front that > they need to change the logging (and hopefully after they read up on > SLF4J, realize that they do have a simple choice to make). I think > that if we packaged a particular slf4j JAR, users will stick with it > when they should be careful and include the implementation best suited > to their environment (i.e. while slf4j-jdk14 could be used probably > without any additional JAR dependencies, it won't work as expected in > Geronimo which I think relies on log4j). > > On Sat, Oct 17, 2009 at 10:08 PM, Shiva Kumar H R <[email protected]> > wrote: > > Thanks Bryant. I just verified that issues 1) and 3) are fixed in trunk. > > > > For 2) shouldn't we make sure that by *default* info messages are *not* > sent > > to console (i.e. by default we either disable logging or direct it some > log > > file)? > > > > Thanks, > > Shiva Kumar H R > > > > On Sat, Oct 17, 2009 at 6:11 PM, Bryant Luk <[email protected]> wrote: > > > >> Hello Shiva, > >> > >> 1) I've fixed on trunk I think. I don't think there were any more > >> additional libraries required but I don't really use Google Docs. > >> > >> For 2), those should just be normal INFO messages. If you disable > >> logging (and replace the slf4j-simple.jar with a real logging > >> framework JAR), then they should disappear. > >> > >> 3) There is an issue with trying to read off the RuntimeContext when > >> outside a client/server invocation. I've put in a fix on trunk as > >> well. IMO, we should probably come back to this in the future though > >> and look at it when all the client side RuntimeContext stuff is > >> decided. > >> > >> On Sat, Oct 17, 2009 at 4:36 AM, Shiva Kumar H R <[email protected]> > >> wrote: > >> > 1) Ant build of examples\client\GoogleDocs-client is failing with the > >> > following error: > >> > > >> > BUILD FAILED > >> > > >> > C:\_Web2.0FeP\apache-wink-1.0-incubating\examples\client\GoogleDocs-client\build > >> > .xml:68: C:\_Web2.0FeP\apache-wink-1.0-incubating\lib\examples not > found. > >> > > >> > 2) While running examples\client\QADefects-client or > >> > examples\client\ReadRSS-client there is an extra output to the console > >> from > >> > within the Wink client library as follows: > >> > > >> > 15 [main] INFO > >> > org.apache.wink.common.internal.application.ApplicationFileLoader > >> > - Loading application from META-INF/core/wink-providers. > >> > 218 [main] INFO org.apache.wink.client.internal.ResourceImpl - Issuing > >> > client GE > >> > T method request to URI at http://localhost:8080/QADefect/rest with > null > >> > entity > >> > class and CaseInsensitiveMultivaluedMap > >> > [map=[Accept=application/atomsvc+xml]] h > >> > eaders > >> > 828 [main] INFO org.apache.wink.client.internal.ResourceImpl - Issuing > >> > client GE > >> > T method request to URI at > http://localhost:8080/QADefect/rest/defectswith > >> > null > >> > entity class and CaseInsensitiveMultivaluedMap > >> > [map=[Accept=application/atom+xm > >> > l]] headers > >> > > >> > 3) Running examples\client\QADefects-client throws a > NullPointerException > >> > towards the end. > >> > > >> > 1218 [main] INFO org.apache.wink.client.internal.ResourceImpl - > Issuing > >> > client G > >> > ET method request to URI at > >> > http://localhost:8080/QADefect/rest/defects/13/tests > >> > with null entity class and CaseInsensitiveMultivaluedMap > >> > [map=[Accept=applicati > >> > on/atom+xml]] headers > >> > ID> urn:com:hp:qadefects:test:8 Title> Check Edit Profile page > >> > java.lang.NullPointerException > >> > at > >> > org.apache.wink.common.internal.model.ModelUtils.readValue(ModelUtils > >> > .java:258) > >> > at > >> > org.apache.wink.common.internal.model.ModelUtils.readValue(ModelUtils > >> > .java:241) > >> > at > >> > org.apache.wink.common.model.atom.AtomContent.getValue(AtomContent.ja > >> > va:366) > >> > at > >> > org.apache.wink.common.model.atom.AtomContent.getValue(AtomContent.ja > >> > va:340) > >> > at > >> > org.apache.wink.common.model.atom.AtomContent.getValue(AtomContent.ja > >> > va:325) > >> > at org.apache.wink.example.QADefectsClient.listTests(Unknown > >> Source) > >> > at org.apache.wink.example.QADefectsClient.main(Unknown Source) > >> > > >> > Thanks, > >> > Shiva Kumar H R > >> > > >> > On Fri, Oct 16, 2009 at 11:24 PM, Bryant Luk <[email protected]> wrote: > >> > > >> >> +1 > >> >> > >> >> On Thu, Oct 15, 2009 at 3:33 PM, Mike Rheinheimer <[email protected] > > > >> >> wrote: > >> >> > +1, binding this time! woohoo! > >> >> > > >> >> > mike > >> >> > > >> >> > > >> >> > On Thu, Oct 15, 2009 at 2:25 PM, Nicholas L Gallardo > >> >> > <[email protected]> wrote: > >> >> >> > >> >> >> I'm calling a vote to release Wink 1.0 > >> >> >> > >> >> >> The Maven staging area is at: > >> >> >> > https://repository.apache.org/content/repositories/wink-staging-002/ > >> >> >> > >> >> >> The distributions are in: > >> >> >> > >> >> > >> > https://repository.apache.org/content/repositories/wink-staging-002/org/apache/wink/apache-wink/1.0-incubating/ > >> >> >> > >> >> >> > >> >> >> This release is tagged at: > >> >> >> > >> >> > >> > https://svn.apache.org/repos/asf/incubator/wink/tags/wink-1.0-incubating/ > >> >> >> > >> >> >> The vote will be open here for at least 72 hours. > >> >> >> > >> >> >> This is my +1 > >> >> >> > >> >> >> > >> >> >> Nicholas Gallardo > >> >> >> WebSphere - REST & WebServices Development > >> >> >> [email protected] > >> >> >> Phone: 512-286-6258 > >> >> >> Building: 903 / 5G-016 > >> >> > > >> >> > >> > > >> > > >
