-
Notifications
You must be signed in to change notification settings - Fork 301
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-16077 Add option to download WU result in CSV format #9056
Conversation
https://track.hpccsystems.com/browse/HPCC-16077 |
@afishbeck please review |
return; | ||
} | ||
|
||
if (!strieq(fieldName, "row")) |
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.
Are you sure you can rely on the fieldname "row" not being used for other types of fields. And is this processing post xpath, where a row might have another name? i.e.
<Records>
<Record>
Rather than
<Records>
<Row>
?
@wangkx I think I need to step through an example before I can continue much further. |
@wangkx testing looked ok when I didn't use xpaths on the datasets.. but I'll wait until you fix that (as we discussed) before continuing to review the code. Here's some code I threw together to test. Take out all the XPATHs and run it once. I think it will work ok. Then run with the xpaths and debug. Notice the xpath('/Name').. that's a different variation then just renaming the 'Row'. If you need help let me know. You can look at the legacy eclwatch view to get a good idea of what it should look like. `namerec := RECORD Addressrec := RECORD bestrec := RECORD Personrec := RECORD Peoplerec := RECORD ds := dataset([{[{{{'A', 'B'},{'C', 'Z'}}, [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}], [{'C1', 'Z1'},{'C2', 'Z2'},{'C3', 'Z2'}], [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}]},{{{'A', 'B'},{'C', 'Z'}}, [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}], [{'C1', 'Z1'},{'C2', 'Z2'},{'C3', 'Z2'}], [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}]},{{{'A', 'B'},{'C', 'Z'}}, [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}], [{'C1', 'Z1'},{'C2', 'Z2'},{'C3', 'Z2'}], [{'Aa', 'Bb'},{'Ab', 'Bc'},{'Ac', 'Bd'}]}]}], Peoplerec); output(ds, named('People'));` |
The latest code should handle both xpath and no xpath. It still needs to be cleaned. |
@afishbeck Please re-review |
@wangkx Let me know when you've finished the changes you mentioned. |
The option can be used by WsWorkunits.WUResultBin.format = 'csv'. Signed-off-by: wangkx <kevin.wang@lexisnexis.com>
@afishbeck the code has been cleaned. Please review. |
Automated Smoketest HPCC Stop: OK |
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.
Don't forget to remind Gordon that it needs to be integrated into the UI.
@GordonSmith or @miguelvazq you may need to add an option to download WU Result csv to ECLWatch UI: https://track.hpccsystems.com/browse/HPCC-16325. @richardkchapman this PR is ready to be merged. |
The option can be used by WsWorkunits.WUResultBin.format = 'csv'.
Signed-off-by: wangkx kevin.wang@lexisnexis.com