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
importccl: implement EXPORT csv #24755
Closed
+1,086
−274
Commits on Apr 23, 2018
-
sql: remove DistLoader wrapper
dt committedApr 10, 2018 I concur with Dan’s year-old TODO: DistLoader doesn’t seem like a useful abstraction and we’re better off just getting an unwrapped DistSQLPlanner into the caller’s hands so they can use its public API without fighting though additional indirection. Indeed, Ideally I’d like to follow this with exporting more of the DistSQL API, with the goal of eventually moving LoadCSV method to a function in importccl defined entirely in terms of the public DistSQL API, though that will obviously require some refactoring and careful thinking about where to draw maintainable lines between these packages. Release note: none.
Verified
This commit was signed with a verified signature.dt David Taylor
GPG key ID: 01D8438FDFAE4E9A Learn about signing commits
Commits on Apr 24, 2018
-
importccl: implement EXPORT CSV
This adds a DistSQL processor that writes whatever it is given as CSV files to a specified storage location. This processor can be appended as a sink for arbitrary SELECT queries. The primary intended use-case is dumping tables as CSV for bulk-export, but it can also be used on any SELECT query, so it could also be used in report generation or other workflows. Release note (enterprise change): prototype support for EXPORT CSV.
Verified
This commit was signed with a verified signature.dt David Taylor
GPG key ID: 01D8438FDFAE4E9A Learn about signing commits