[
https://issues.apache.org/jira/browse/PHOENIX-2612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15110161#comment-15110161
]
Hudson commented on PHOENIX-2612:
---------------------------------
SUCCESS: Integrated in Phoenix-master #1096 (See
[https://builds.apache.org/job/Phoenix-master/1096/])
PHOENIX-2612 RuleGeneartorTest updated with right pattern (Karan (jtaylor: rev
dab79cbe0e4d10e1c259198de8e95b7e93d66b57)
* phoenix-pherf/src/test/java/org/apache/phoenix/pherf/ResourceTest.java
* phoenix-pherf/src/main/java/org/apache/phoenix/pherf/PherfConstants.java
* phoenix-pherf/src/test/java/org/apache/phoenix/pherf/RuleGeneratorTest.java
> Pherf - RuleGeneartor test in phoenix-pherf passes and fails in different
> environments.
> -----------------------------------------------------------------------------------------
>
> Key: PHOENIX-2612
> URL: https://issues.apache.org/jira/browse/PHOENIX-2612
> Project: Phoenix
> Issue Type: Bug
> Affects Versions: 4.6.0
> Reporter: Karan Singhal
> Assignee: Karan Singhal
> Labels: Performance
> Fix For: 4.7.0
>
> Attachments: PHOENIX-2612.1.patch, PHOENIX-2612.patch
>
>
> This is happening because regex used in test for searching path of
> test-scenario.xml is not specific. The pattern being used is
> *scenario.*scenario.*.xml due to which pattern matcher finds all the
> different xmls in the package. After that all the xml's are parsed if
> pattern matcher finds test-scenario.xml in path first then test-scenario.xml
> is parsed first and test passes because test is always reading datamodel at
> index 0 (hard coded in test) from list of datamodels and if any other xml is
> parsed first it fails because it did not get right datamodel.
> Solution is simple just make matcher scenario in test to be more specific so
> that every time test-scenario.xml is picked up. After looking at some other
> tests I found .*test_scenario.xml is used in lot of other tests which is the
> most appropriate for searching test_sceanario.xml and can be used for this
> test too.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
