[ https://issues.apache.org/jira/browse/PHOENIX-2368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15088742#comment-15088742 ]
ramkrishna.s.vasudevan commented on PHOENIX-2368: ------------------------------------------------- With the patch applied I tried running the test {code} Running org.apache.phoenix.query.ConnectionlessTest Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.581 sec - in org.apache.phoenix.query.ConnectionlessTest Results : Tests run: 3, Failures: 0, Errors: 0, Skipped: 0 {code} And seems to pass. So this QA error is not true? Shall I go ahead with the commit? > Call Driver.accept() in Driver.getConnection and return null if false > --------------------------------------------------------------------- > > Key: PHOENIX-2368 > URL: https://issues.apache.org/jira/browse/PHOENIX-2368 > Project: Phoenix > Issue Type: Bug > Reporter: James Taylor > Assignee: ramkrishna.s.vasudevan > Labels: SFDC, newbie > Fix For: 4.7.0 > > Attachments: PHOENIX-2368.patch, PHOENIX-2368.patch, > PHOENIX-2368_1.patch > > > The driver should return "null" if it realizes it is the wrong kind of driver > to connect to the given URL. > Your driver is throwing an exception instead of returning null. > Here's the start of a test case: > {code} > public static void main(String[] args) throws Exception { > Class.forName(PhoenixDriver.class.getName()); > PhoenixDriver d = (PhoenixDriver) DriverManager.getDriver("jdbc:phoenix"); > d.close(); > DriverManager.getConnection("any text whatever you want to put here"); > } > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)