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