[ 
https://issues.apache.org/jira/browse/PHOENIX-881?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13942425#comment-13942425
 ] 

James Taylor commented on PHOENIX-881:
--------------------------------------

We shouldn't need the clearCache() in BaseConnectedQueryIT#deletePriorTables as 
it's called during initialization time - can you remove that, or do tests fail 
without it?

The Zookeeper quorum may contain a ":"? That would cause problems in Phoenix, 
as the parsing doesn't expect/allow that. Is that a WINDOWS thing? We should 
file a JIRA for that - I don't think it's a blocker b/c as you mentioned, in 
these cases you can put the info in the hbase-sites.xml.

It's difficult for me to evaluate the MetaDataEndPointImpl change as I'm not 
seeing it in the context of the code. Can you spin up a pull request? Or maybe 
just copy/paste the entire function here? Is that in loadTable()? Was your diff 
done against the 4.0 branch? We should apply it to both, right? 

 

> end2end test fails to run against a live cluster using end2endtest.py
> ---------------------------------------------------------------------
>
>                 Key: PHOENIX-881
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-881
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 4.0.0
>            Reporter: Jeffrey Zhong
>            Assignee: Jeffrey Zhong
>         Attachments: phoenix-881.patch
>
>
> In most IT tests, we're using constant PHOENIX_JDBC_URL instead of 
> BaseConnectedQueryIT#getUrl() which will return correct url depending where 
> the test is running against. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to