CodingCat commented on pull request #30972:
URL: https://github.com/apache/spark/pull/30972#issuecomment-752696792


   > Isn't it a bug in Univocity? The pointers you show say it's just about 
string caching instead of limiting the size of value or header.
   
   I thought so, the logic there is very counter-intuitive. However it is hard 
to say if the author intentionally did this, since in 
[NormalizedString](https://github.com/uniVocity/univocity-parsers/blob/e09114c6879fa6c2c15e7365abc02cda3e193ff7/src/main/java/com/univocity/parsers/common/NormalizedString.java)
 , stringCache is used as the gatewayfor all valueOfxxxx calls, seems the 
author was trying to impose the max string in header (the following screenshot 
shows that, stringCache seems only involved for header operations? )
   
   
![image](https://user-images.githubusercontent.com/678008/103369986-738fb800-4a80-11eb-82f7-3b0739ec1580.png)
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to