----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7002/#review11382 -----------------------------------------------------------
src/java/org/apache/hcatalog/listener/NotificationListener.java <https://reviews.apache.org/r/7002/#comment24422> It will be good to have root-caused NPE problem of Datanucleus here, but that really is a topic of separate jira. For now, I will suggest to add a note in comment that some of the information is lost for a partition object that is sent over message bus. src/java/org/apache/hcatalog/listener/NotificationListener.java <https://reviews.apache.org/r/7002/#comment24423> Same as above. Add a note in comment that some of the information is lost in table object that is sent on message bus. - Ashutosh Chauhan On Sept. 11, 2012, 4:17 a.m., Travis Crawford wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7002/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2012, 4:17 a.m.) > > > Review request for hcatalog and Travis Crawford. > > > Description > ------- > > Fix broken TestNotificationListener. > > > This addresses bug HCATALOG-498. > https://issues.apache.org/jira/browse/HCATALOG-498 > > > Diffs > ----- > > src/java/org/apache/hcatalog/listener/NotificationListener.java 0660042 > src/test/org/apache/hcatalog/listener/TestNotificationListener.java 58ef72c > src/test/org/apache/hcatalog/mapreduce/HCatBaseTest.java 4302e67 > > Diff: https://reviews.apache.org/r/7002/diff/ > > > Testing > ------- > > > Thanks, > > Travis Crawford > >
