[ https://issues.apache.org/jira/browse/DBUTILS-88?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13210115#comment-13210115 ]
Moandji Ezana commented on DBUTILS-88: -------------------------------------- Looks good to me. It might be worth marking the other constructors as @Deprecated, too. If AsyncQueryRunner is about wrapping a QueryRunner, AsyncQueryRunner(ExecutorService executorService, QueryRunner queryRunner) best expresses that. Ultimately, it might be worth making QueryRunner an interface that is implemented by both AsyncQueryRunner and the current QueryRunner (renamed to BasicQueryRunner?). > Make AsyncQueryRunner be a decorator around a QueryRunner > --------------------------------------------------------- > > Key: DBUTILS-88 > URL: https://issues.apache.org/jira/browse/DBUTILS-88 > Project: Commons DbUtils > Issue Type: Task > Reporter: Moandji Ezana > Priority: Minor > Attachments: AsyncQueryRunner_wraps_QueryRunner.txt, > DBUTILS-88v1.patch > > > AsyncQueryRunner duplicates much of the code in QueryRunner. Would it be > possible for AsyncQueryRunner to simply decorate a QueryRunner with async > functionality, in the same way a BufferedInputStream might decorate an > InputStream? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira