[GitHub] kaspersorensen commented on a change in pull request #209: Added the DynamoDB module to the `full` module. And created DataContextFactory for it.

kaspersorensen commented on a change in pull request #209: Added the DynamoDB module to the `full` module. And created DataContextFactory for it. URL: https://github.com/apache/metamodel/pull/209#discussion_r259604835 ## File path: dynamodb/src/main/resources/META-INF/services/org.

[GitHub] kaspersorensen commented on a change in pull request #209: Added the DynamoDB module to the `full` module. And created DataContextFactory for it.

kaspersorensen commented on a change in pull request #209: Added the DynamoDB module to the `full` module. And created DataContextFactory for it. URL: https://github.com/apache/metamodel/pull/209#discussion_r259604829 ## File path: dynamodb/src/main/java/org/apache/metamodel/dynamo

Re: Pool + timer threads in JdbcCompiledQuery

HI Dana, The CompiledQuery is intended for reuse, so the fact that you re-create it for every query is the crux of the issue here. If you don't intend to reuse the query, then you don't have any benefit in compiling it. So your solutions could either be: (1) Don't use compiled query, just execute