You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by mattyb149 <gi...@git.apache.org> on 2018/04/17 14:48:33 UTC

[GitHub] nifi issue #2638: NIFI-5082: Added support for custom Oracle timestamp types...

Github user mattyb149 commented on the issue:

    https://github.com/apache/nifi/pull/2638
  
    I REALLY didn't want to have to do this explicit Oracle stuff in the generic code, but the other options I tried weren't viable. I tried isolating the SQL type conversion to the DatabaseAdapters, but ExecuteSQL uses JdbcCommon and does not use DatabaseAdapters, so the problem would still exist. I also tried calling getTimestamp() in the default handling block (before attempting to store a String) but that is really brittle since other drivers may or may not handle getTimestamp() in a manner we could rely on. This is a simple straightforward (but dirty) fix.


---