[jira] [Commented] (DBUTILS-99) Create QueryRunner Factory
[ https://issues.apache.org/jira/browse/DBUTILS-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13437904#comment-13437904 ] Moandji Ezana commented on DBUTILS-99: -- I'm also curious what you mean. Like Simone, I've found it easy to use QueryRunners with Guice. Create QueryRunner Factory -- Key: DBUTILS-99 URL: https://issues.apache.org/jira/browse/DBUTILS-99 Project: Commons DbUtils Issue Type: Improvement Affects Versions: 1.5 Reporter: William R. Speirs Assignee: William R. Speirs Priority: Minor Currently there is no good way to use dependency injection with QueryRunners. Creating a simple factory class to allow for easy injection (and therefore easier unit test creation) should solve this problem. -- 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
[jira] [Commented] (DBUTILS-98) Add missing JavaDoc to QueryRunner#insert
[ https://issues.apache.org/jira/browse/DBUTILS-98?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13437909#comment-13437909 ] Moandji Ezana commented on DBUTILS-98: -- Hi Simone, Sorry about the rookie mistakes. I'll blame #1 on late night inattention. I have no excuse for #2 ;). Add missing JavaDoc to QueryRunner#insert - Key: DBUTILS-98 URL: https://issues.apache.org/jira/browse/DBUTILS-98 Project: Commons DbUtils Issue Type: Task Reporter: Moandji Ezana Assignee: Simone Tripodi Priority: Minor Fix For: 1.6 Attachments: QueryRunner_insert_add_javadoc.patch -- 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
[jira] [Commented] (DBUTILS-99) Create QueryRunner Factory
[ https://issues.apache.org/jira/browse/DBUTILS-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13438268#comment-13438268 ] Moandji Ezana commented on DBUTILS-99: -- I use constructor injection with Guice as well. Why can't you just do: public MyClass(QueryRunner queryRunner) Unit tests could still inject whatever instance they like. Maybe I'm missing your point. What would the factory's API be? Create QueryRunner Factory -- Key: DBUTILS-99 URL: https://issues.apache.org/jira/browse/DBUTILS-99 Project: Commons DbUtils Issue Type: Improvement Affects Versions: 1.5 Reporter: William R. Speirs Assignee: William R. Speirs Priority: Minor Currently there is no good way to use dependency injection with QueryRunners. Creating a simple factory class to allow for easy injection (and therefore easier unit test creation) should solve this problem. -- 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
[jira] [Created] (DBUTILS-98) Add JavaDoc to QueryRunner#insert
Moandji Ezana created DBUTILS-98: Summary: Add JavaDoc to QueryRunner#insert Key: DBUTILS-98 URL: https://issues.apache.org/jira/browse/DBUTILS-98 Project: Commons DbUtils Issue Type: Task Reporter: Moandji Ezana Fix For: 1.6 -- 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
[jira] [Updated] (DBUTILS-98) Add JavaDoc to QueryRunner#insert
[ https://issues.apache.org/jira/browse/DBUTILS-98?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Moandji Ezana updated DBUTILS-98: - Attachment: QueryRunner_insert_add_javadoc.patch QueryRunner_insert_add_javadoc.patch adds the missing documentation. Add JavaDoc to QueryRunner#insert - Key: DBUTILS-98 URL: https://issues.apache.org/jira/browse/DBUTILS-98 Project: Commons DbUtils Issue Type: Task Reporter: Moandji Ezana Fix For: 1.6 Attachments: QueryRunner_insert_add_javadoc.patch -- 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
[jira] [Updated] (DBUTILS-98) Add JavaDoc to QueryRunner#insert
[ https://issues.apache.org/jira/browse/DBUTILS-98?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Moandji Ezana updated DBUTILS-98: - Priority: Minor (was: Major) Add JavaDoc to QueryRunner#insert - Key: DBUTILS-98 URL: https://issues.apache.org/jira/browse/DBUTILS-98 Project: Commons DbUtils Issue Type: Task Reporter: Moandji Ezana Priority: Minor Fix For: 1.6 Attachments: QueryRunner_insert_add_javadoc.patch -- 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
[jira] [Updated] (DBUTILS-87) Return generated key on insert
[ https://issues.apache.org/jira/browse/DBUTILS-87?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Moandji Ezana updated DBUTILS-87: - Attachment: AsyncQueryRunner_add_insert_methods.patch AsyncQueryRunner_add_insert_methods.patch adds insert methods to AsyncQueryRunner and unit tests verify that call is delegated to the wrapped QueryRunner. Return generated key on insert -- Key: DBUTILS-87 URL: https://issues.apache.org/jira/browse/DBUTILS-87 Project: Commons DbUtils Issue Type: New Feature Reporter: Moandji Ezana Assignee: William R. Speirs Attachments: AsyncQueryRunner_add_insert_methods.patch, QueryRunner_insert.txt It would be useful to have an insert method on QueryRunner that returns the id of the new record. -- 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