[GitHub] [nifi-minifi-cpp] bakaid commented on issue #656: MINIFI-1013 Used soci library.

[GitBox <gi...@apache.org>]

bakaid commented on issue #656: MINIFI-1013 Used soci library.
URL: https://github.com/apache/nifi-minifi-cpp/pull/656#issuecomment-576349582
 
 
   @am-c-p-p I have tested the functionality of the PR on the following branch: https://github.com/bakaid/nifi-minifi-cpp/commits/minifi-1013-soci-test
   On this I rebased your branch to the current master (as the third party refactor has been merged since, at it brought a lot of changes), and added a commit (https://github.com/bakaid/nifi-minifi-cpp/commit/7002285c050f24988ef3823bd5a9c4ad759347dc), that now *really* fixes the compilation on all platforms. Please adapt your branch to mine, if you agree with the changes.
   I have been able to verify QueryDatabaseTable and ExecuteSQL functionality on macOS, Linux and Windows.
   We still have to find a solution for the dt_date issues. My current best solution would be to just document this issue in the processor documentation and add a follow-up task for it.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services