Github user HeartSaVioR commented on the issue:

    https://github.com/apache/spark/pull/23169
  
    @DaveDeCaprio 
    
    You might miss to roll back change in test.
    
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99632/testReport/org.apache.spark.sql.catalyst.trees/TreeNodeSuite/treeString_limits_plan_length/
    
    I also think you need to add a new test with setting configuration to some 
value and see whether it works properly.


---

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

Reply via email to