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-17491 Add ecl commands to import/export entire querysets #10542
Conversation
https://track.hpccsystems.com/browse/HPCC-17491 |
3fd0afb
to
41c42b1
Compare
ecl queries export <target> --output=file ecl queries import <target> <file> [--clone-active-state][--replace] Signed-off-by: Anthony Fishbeck <anthony.fishbeck@lexisnexis.com>
@afishbeck Who can review this? |
Perhaps @wangkx, can you review? |
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.
@afishbeck a few minor comments
name.append("_activeonly_"); | ||
CDateTime dt; | ||
dt.setNow(); | ||
dt.getString(name, true); |
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.
Where do you use the name?
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.
oops forgot to set it to optFIlename. Will change.
ecl/eclcmd/queries/ecl-queries.cpp
Outdated
if (optDestQuerySet.isEmpty()) | ||
optDestQuerySet.set(arg); | ||
else if (optFilename.isEmpty()) | ||
optFilename.set(arg); |
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.
minor: wrong indent
esp/scm/ws_workunits.ecm
Outdated
@@ -1997,6 +2053,8 @@ ESPservice [ | |||
ESPmethod [cache_seconds(60), resp_xsl_default("/esp/xslt/WUQuerysetQueries.xslt")] WUQuerysetDetails(WUQuerySetDetailsRequest, WUQuerySetDetailsResponse); | |||
ESPmethod [cache_seconds(60), resp_xsl_default("/esp/xslt/WUQueryDetails.xslt")] WUQueryDetails(WUQueryDetailsRequest, WUQueryDetailsResponse); | |||
ESPmethod [cache_seconds(60)] WUMultiQuerysetDetails(WUMultiQuerySetDetailsRequest, WUMultiQuerySetDetailsResponse); | |||
ESPmethod WUQuerysetImport(WUQuerysetImportRequest, WUQuerysetImportResponse); | |||
ESPmethod WUQuerysetExport(WUQuerysetExportRequest, WUQuerysetExportResponse); |
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.
Should we add the min_ver so that we can trace the change?
updateFlags |= DALI_UPDATEF_APPEND_CLUSTER; | ||
|
||
cloner.enableFileCloning(updateFlags, req.getDfsServer(), process.str(), req.getSourceProcess(), req.getAllowForeignFiles()); | ||
} |
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.
Should we throw an exception if the target is not a roxie?
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.
I'll move some code around and validate at the top of the method
Signed-off-by: Anthony Fishbeck <anthony.fishbeck@lexisnexis.com>
@wangkx updated please review |
@AttilaVamos could the above build errors be an anomaly? Nothing I changed should affect javaembed.cpp. |
@afishbeck It is not your fault. Master is fixed and your PR rescheduled. |
Automated Smoketest: ✅
Install hpccsystems-platform-community_6.5.0-trunk0.el7.x86_64.rpm Unit tests result:
Regression test result:
HPCC Stop: OK |
@wangkx Please review |
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.
@afishbeck looks good.
ecl queries export --output=file
ecl queries import [--clone-active-state][--replace]
Signed-off-by: Anthony Fishbeck anthony.fishbeck@lexisnexis.com
Type of change:
Checklist:
Testing: