events.action: opened
events.author: HyukjinKwon
events.text: ### What changes were proposed in this pull request?
  
  This PR proposes to clean up `UnivocityParser`. 
  
  ### Why are the changes needed?
  
  It will slightly improve the performance since we don't do unnecessary computation for Array concatenations/creation.
  
  ### Does this PR introduce any user-facing change?
  
  No.
  
  ### How was this patch tested?
  
  Manually ran the existing tests.
events.title: [SPARK-30530][SQL][FOLLOW-UP] Remove unnecessary codes and fix comments accordingly in UnivocityParser
events.type: issue
events.action: created
events.author: HyukjinKwon
events.text: cc @cloud-fan and @MaxGekk. It's rather minor but wanted to clean up here.
events.type: comment
events.action: created
events.author: HyukjinKwon
events.text: retest this please
events.type: comment
events.action: created
events.author: HyukjinKwon
events.text: retest this please
events.type: comment
events.action: created
events.author: MaxGekk
events.text: Something wrong is going on. This exception shouldn't happen:
  ```
  Caused by: java.lang.IllegalStateException: Number of logging event reached the limit: 100
  	at org.apache.spark.SparkFunSuite$LogAppender.append(SparkFunSuite.scala:197)
  	at org.apache.log4j.AppenderSkeleton.doAppend(AppenderSkeleton.java:251)
  ```
  It was added by https://github.com/apache/spark/pull/27166, and the limit shouldn't be reachable.
events.type: comment
events.action: created
events.author: MaxGekk
events.text: I propose to assign names to log appenders, and print the names in exception. So, we will know which one wasn't removed.
events.type: comment
events.action: created
events.author: srowen
events.text: Jenkins, retest this please
events.type: comment
events.action: created
events.author: HyukjinKwon
events.text: Merged to master.
events.type: comment
data_with_bots.bot: **[Test build #117097 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117097/testReport)** for PR 27287 at commit [`c2c5ab0`](https://github.com/apache/spark/commit/c2c5ab02475eff2105ab6bee2b93f3fc2eee693c).
data_with_bots.meta: This message directly preceded comment from bot *SparkQA* by author *HyukjinKwon*:
data_with_bots.text: retest this please
data_with_bots.bot: **[Test build #117121 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117121/testReport)** for PR 27287 at commit [`c2c5ab0`](https://github.com/apache/spark/commit/c2c5ab02475eff2105ab6bee2b93f3fc2eee693c).
data_with_bots.meta: This message directly preceded comment from bot *SparkQA* by author *HyukjinKwon*:
data_with_bots.text: retest this please
data_with_bots.bot: **[Test build #117124 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117124/testReport)** for PR 27287 at commit [`c2c5ab0`](https://github.com/apache/spark/commit/c2c5ab02475eff2105ab6bee2b93f3fc2eee693c).
data_with_bots.meta: This message directly preceded comment from bot *SparkQA* by author *srowen*:
data_with_bots.text: Jenkins, retest this please
data_without_bots.meta: This message directly preceded comment from bot *SparkQA* by author *HyukjinKwon*:
data_without_bots.text: retest this please
data_without_bots.meta: This message directly preceded comment from bot *SparkQA* by author *HyukjinKwon*:
data_without_bots.text: retest this please
data_without_bots.meta: This message directly preceded comment from bot *SparkQA* by author *srowen*:
data_without_bots.text: Jenkins, retest this please