Github user mgaido91 commented on the issue:

    https://github.com/apache/spark/pull/19916
  
    my only concern about this PR and about removing `addReferenceObj` is that 
we are saving a cast every time. I am not sure about how much this can affect 
performances, but I am planning to make some tests. Actually after this patch, 
the only place where it is used is for metric terms.
    WDYT?


---

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

Reply via email to