Github user mgaido91 commented on the issue:

    https://github.com/apache/spark/pull/19916
  
    @cloud-fan I think a lot of caller-side changes would be needed as well, 
since now we are not passing any variable name when we are referencing with 
`addReferenceMinorObj`. I am not sure that adding a new argument to 
`addReferenceMinorObj` only for the comment is the right way to go.
    
    Sorry for this additional comment, I just want to make sure that I am 
providing you all the details to choose the best option. I swear that now if 
you say, let's add the comment, I'll do.


---

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

Reply via email to