diff options
author | ToineSiebelink <toine.siebelink@est.tech> | 2022-12-22 14:47:07 +0000 |
---|---|---|
committer | ToineSiebelink <toine.siebelink@est.tech> | 2022-12-22 15:31:36 +0000 |
commit | ded9f06f42bbe751bdec3d763ae5216d76df82c4 (patch) | |
tree | d8bd1857a681314660f09315be14c7a51bde5a06 /cps-path-parser | |
parent | dbf10db6f468075293d61e7bbeb9006fd15cfce6 (diff) |
Temp Table Creation improvements
- extracted methods for more generic temp table creation
- optimized
- added perf test on use case that depends on temp table creation
- had some doubts about stopwatch use in recent tests, but was all OK, just renamed some
Issue-ID: CPS-1422
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I22cabb9b0ba1b0aa8576a2d756d77af46eebc1b0
Diffstat (limited to 'cps-path-parser')
-rw-r--r-- | cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathUtilSpec.groovy | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathUtilSpec.groovy b/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathUtilSpec.groovy index df2e9d72c6..f1a878d63b 100644 --- a/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathUtilSpec.groovy +++ b/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathUtilSpec.groovy @@ -74,16 +74,16 @@ class CpsPathUtilSpec extends Specification { def 'CPS Path Processing Performance Test.'() { when: '200,000 paths are processed' - def setupStopWatch = new StopWatch() - setupStopWatch.start() + def stopWatch = new StopWatch() + stopWatch.start() (1..100000).each { CpsPathUtil.getNormalizedXpath('/long/path/to/see/if/it/adds/paring/time/significantly/parent/child[@common-leaf-name="123"]') CpsPathUtil.getNormalizedXpath('//child[@other-leaf=1]/leaf-name[text()="search"]/ancestor::parent') } - setupStopWatch.stop() + stopWatch.stop() then: 'it takes less then 10,000 milliseconds' // In CI this actually takes about 3-5 sec which is approx. 50+ parser executions per millisecond! - assert setupStopWatch.getTotalTimeMillis() < 10000 + assert stopWatch.getTotalTimeMillis() < 10000 } } |