-
Notifications
You must be signed in to change notification settings - Fork 2k
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
GH-15972: Add Option Filtering File System For Reading and Writing #16057
Conversation
@krasinski The PR looks good to me, good job! Can you also add some docs.? |
throw new IllegalArgumentException("Error parsing file_deny_glob parameter"); | ||
} | ||
} | ||
else if(s.matches("embedded")) { |
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.
@krasinski : This seems to significantly slow down H2O. Is it possible to do this check less frequently? Like when there is a actual reading/writing that is about to happen rather than any time someone wants to look at the key?
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.
@krasinski : It seems like my previous comment was wrong. The failure is probably caused by error throws by worker node not caught and the whole test just hang. Here is a copy of the test run time:
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.
Looks good to me, thank you @krasinski !
02cb709
fix #15972