[GitHub] nifi issue #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logica...

2017-05-11 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1142 Just wanted to share my progress on this. I've add Logical Decimal support as well as Date/Time/Timestamp. As @ToivoAdams pointed out, it's important to make this enhancement work with

[GitHub] nifi issue #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logica...

2017-05-02 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1142 @trixpan Thanks for sharing the comment. @ToivoAdams @joewitt Sorry for not responding, I hadn't watched the JIRA issue and didn't notice this has been updated. I will polish my commit and do

[GitHub] nifi issue #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logica...

2017-05-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1142 Adding @ToivoAdams comment here as his last comment on JIRA did use github's @ mentions instead of JIRAs [~ ] mentions: > @ijokarumawak > > Sorry I didn't have time to look your

[GitHub] nifi issue #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logica...

2017-02-14 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/1142 @ijokarumawak @ToivoAdams @trixpan can we reach consensus on next steps here and go for it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi issue #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logica...

2017-01-23 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1142 Hi @ToivoAdams @trixpan I've encountered similar issue. So I've researched and tried to use Avro 1.8.1 LogicalTypes, without knowing there's an existing JIRA and PR to solve that,