Github user nico-pappagianis commented on the issue:
https://github.com/apache/phoenix/pull/221
Looks like everything's there.
Oh sorry about the white space - I must have hit IntelliJ auto format
shortcut.
Thanks again for all the help!
On Thursday, November 10, 2016, Josh Mahonin <[email protected]>
wrote:
> Thanks @nico-pappagianis <https://github.com/nico-pappagianis> I think
> I've got it squared away. Your changes are contained in the following
> commits:
>
> 3886638
>
<https://github.com/apache/phoenix/commit/3886638b04a3e494be70e1e681ad56cb1c7e33c5>
> 3b07c2f
>
<https://github.com/apache/phoenix/commit/3b07c2f1bad96210e9a81079aed1149f5fcb71bc>
>
> Please let me know if I missed anything. Note that I pruned some of the
> whitespace-only changes to unrelated files. The unit tests pass for me
> though. Thanks and well done!
>
> â
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/phoenix/pull/221#issuecomment-259846577>, or
mute
> the thread
>
<https://github.com/notifications/unsubscribe-auth/ABdNQLwI-Sa8KBNHaQQDKoRJYfxMa53Cks5q87OmgaJpZM4Krmym>
> .
>
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
