spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From srowen <...@git.apache.org>
Subject [GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...
Date Sun, 18 Sep 2016 15:57:16 GMT
Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/15138
  
    I think you know this area better than anyone. The update seems reasonable. So the only
substantive change is to not limit the maximum length of a line? I tend to agree with removing
arbitrary limits (especially if one can still set them if desired) but was this set for a
particular reason before, like, guarding against someone parsing non-CSV and chewing up a
load of memory?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message