New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HPCC-22396 Arrange WU Analysis for automated regression #12728
Conversation
https://track.hpccsystems.com/browse/HPCC-22396 |
@shamser These test cases won't be run automatically with Regression Suite in Smoketest and OBT, based on they are not in the default (testing/regress/ecl/) directory. |
@AttilaVamos I know. This is the regression that I am trying to make work with the regression suite. There will be a few dozen WU Analysis regressions in the testing/wuanalysis subdirectory. I can execute with ./ecl-test --suiteDir ../wuanalysis run For each query, there are 2 versions. The first version of each query is a query that demonstrates an issue with the graph. After the execution of the first version of the query, the WU Analysis writes the Analysis in the exceptions/messages section of the WU. The second version reads back the exceptions/messages to check that the issues have been identified. The output to the 2 versions are different and so require 2 different keys. Unfortunately, I cannot do this with just one version as the analysis will not be written out until the first query completes. This is why we need the 2 versions to compare with 2 different keys. This is the reason why I created the request https://track.hpccsystems.com/browse/HPCC-22418. |
edac8e8
to
9a29a6d
Compare
I think this is not the versioning what we have implemented and expects same output for all versions. In your case I can see two separated query one for doing nasty thing and the second which one is checking the nastiness of previous. I mean instead of versioned anahashdistrib1.ecl you have two separated queries:
With these separated ECL file you have two separated functionality, two separated keyfile and everything will be fine, |
Shamser, it would be good to target this to 7.4 - it will not break anything, and would mean we could start running the analysis regression on 7.4 code. |
Signed-off-by: Shamser Ahmed <shamser.ahmed@lexisnexis.co.uk>
@AttilaVamos @ghalliday Please can you review. |
Ticket to execute WU Analysis regression as part of OBT: https://track.hpccsystems.com/browse/HPCC-22430 |
@shamser This logic, file placements and hierarchy seems good and will work if we don't use --PQ in OBT to execute these queries. |
|
||
visits := DATASET(testfile1, layout_visits, THOR); | ||
IMPORT $.common.Files as Files; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
picky: possibly simpler to say
import Files from $.common;
but fine as it stands
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
Signed-off-by: Shamser Ahmed shamser.ahmed@lexisnexis.co.uk
Type of change:
Checklist:
Smoketest:
Testing: