Geez, that's embarrassing. Sorry folks I introduced that bug then disappeared.
The fix looks correct, but as I wrote a while ago, we should look to remove Java serialization from Pirk to avoid any number of potential issues. I will raise a JIRA issue to do this, and expect to schedule it for after the initial Pirk release. Regards, Tim On 02/08/16 20:10, smarthi wrote: > Github user smarthi commented on the issue: > > https://github.com/apache/incubator-pirk/pull/44 > > +1 lgtm > > > --- > If your project is set up for it, you can reply to this email and have your > reply appear on GitHub as well. If your project does not have this feature > enabled and wishes so, or if the feature is enabled but not working, please > contact infrastructure at infrastruct...@apache.org or file a JIRA ticket > with INFRA. > --- >