FiloDB icon indicating copy to clipboard operation
FiloDB copied to clipboard

fix(downsample): wrap all s3-export row keys/values in double-quotes

Open alextheimer opened this issue 1 year ago • 0 comments

Pull Request checklist

  • [x] The commit(s) message(s) follows the contribution guidelines ?
  • [x] Tests for the changes have been added (for bug fixes / features) ?
  • [ ] Docs have been added / updated (for bug fixes / features) ?

Currently, keys/values are wrapped in either single- or double-quotes depending on whether-or-not the string contains any commas. This PR instead wraps all keys/values in double-quotes. Additionally, only unescaped double-quotes are escaped before each string is surrounded by double-quotes.

alextheimer avatar Mar 26 '23 23:03 alextheimer