[
https://issues.apache.org/jira/browse/PHOENIX-2368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15088742#comment-15088742
]
ramkrishna.s.vasudevan edited comment on PHOENIX-2368 at 1/8/16 5:46 AM:
-------------------------------------------------------------------------
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?
[~jamestaylor]?
was (Author: ram_krish):
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)
