> On April 6, 2014, 8:55 a.m., David Chen wrote: > > tajo-docs/src/main/sphinx/table_management/csv.rst, line 59 > > <https://reviews.apache.org/r/20060/diff/1/?file=549965#file549965line59> > > > > "Be careful when using ``\n`` as the field delimiter because CSV uses > > ``\n`` as the line delimiter." > > > > Just wondering: does the CSV format disallow using the same delimiter > > character for fields and lines?
Tajo still allows to use the same delimiter character for fields and lines. It would be better to disallow the same character in the system. What do you think? - Hyunsik ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20060/#review39636 ----------------------------------------------------------- On April 6, 2014, 2:28 a.m., Hyunsik Choi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20060/ > ----------------------------------------------------------- > > (Updated April 6, 2014, 2:28 a.m.) > > > Review request for Tajo. > > > Bugs: TAJO-736 > https://issues.apache.org/jira/browse/TAJO-736 > > > Repository: tajo > > > Description > ------- > > Jinho and I wrote some user documentations for file formats. This patch > contains documentations for CSV file, RCFile, and Parquet file. > > > Diffs > ----- > > tajo-docs/src/main/sphinx/partitioning/column_partitioning.rst > e88d23ff110e0326a667b51c45d5897d37eb04bc > tajo-docs/src/main/sphinx/partitioning/intro_to_partitioning.rst > bfb555f6a49b91adf89f7c7a94ad9226d641e0d0 > tajo-docs/src/main/sphinx/table_management/csv.rst > c11a34315196c276c35ae5eb94c2b66653383b8a > tajo-docs/src/main/sphinx/table_management/parquet.rst > a994b7e4008017d45622c845a19f2884ed15bc8f > tajo-docs/src/main/sphinx/table_management/rcfile.rst > 21f825313a84cffa95e615206108f455817836e8 > > Diff: https://reviews.apache.org/r/20060/diff/ > > > Testing > ------- > > > Thanks, > > Hyunsik Choi > >
