-
Notifications
You must be signed in to change notification settings - Fork 30
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
[3QAbWQds] apoc.export.arrow.all ignores export file config #349
Conversation
// since it says "... use the `{stream:true}` config", but with arrow procedures the streaming mode is implemented via different procedures | ||
if (!apocConfig().getBoolean(APOC_EXPORT_FILE_ENABLED)) { | ||
throw new RuntimeException(EXPORT_TO_FILE_ARROW_ERROR); | ||
} |
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.
Do we not have restrictions on where the file can be written to as well?
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.
Indeed the procedure is not so tested about these cases.
I added an ExportArrowSecurityTest
file
similar to ExportCoreSecurityTest
, to test all these cases.
I preferred to create a separate file, instead of adding the procedures in ExportCoreSecurityTest
,
because it works too differently (parameter order, {stream:true}
not present, etc..)
3e3119e
to
108576f
Compare
108576f
to
3d34341
Compare
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.
LGTM :) Thanks!
* [3QAbWQds] apoc.export.arrow.all ignores export file config * [3QAbWQds] Added ExportArrowSecurityTest and removed some unnecessary tests
…oc#349) * [3QAbWQds] apoc.export.arrow.all ignores export file config * [3QAbWQds] Added ExportArrowSecurityTest and removed some unnecessary tests
…oc#349) * [3QAbWQds] apoc.export.arrow.all ignores export file config * [3QAbWQds] Added ExportArrowSecurityTest and removed some unnecessary tests
Added check for procedures using file mode