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

SaveAsText documentation doesn't address scope of separator argument #2512

Open
rogancarr opened this Issue Feb 12, 2019 · 0 comments

Comments

Projects
None yet
1 participant
@rogancarr
Copy link
Contributor

rogancarr commented Feb 12, 2019

SaveAsText in the Data catalog has a separator parameter, which is specified as a char. However, this throws an ArgumentOutOfRangeException if the separator is not a "space, tab, comma, semicolon, or bar".

Currently, the only way to discover this delimiter limitation is to try and fail.

We have a few options. We could allow any character to be used as a delimiter, we could add documentation to the parameter that says it must be one of these characters, or, if we really do want to scope to a limited set of characters, then we could add a compile-time check by changing this to an enum.

@rogancarr rogancarr added this to To do in Project 13 via automation Feb 12, 2019

@rogancarr rogancarr added the api label Feb 12, 2019

@shauheen shauheen added this to To do in v0.11 Feb 12, 2019

@shauheen shauheen removed this from To do in Project 13 Feb 22, 2019

@shauheen shauheen removed this from To do in v0.11 Feb 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment