Not after SPARK-14642, right?

On Thu, Sep 8, 2016 at 5:07 PM, Reynold Xin <r...@databricks.com> wrote:
> There is a package called scala.
>
>
> On Friday, September 9, 2016, Hyukjin Kwon <gurwls...@gmail.com> wrote:
>>
>> I was also actually wondering why it is being written like this.
>>
>> I actually took a look for this before and wanted to fix them but I found
>> https://github.com/apache/spark/pull/12077/files#r58041468
>>
>> So, I kind of persuaded myself that committers already know about it and
>> there is a reason for this.
>>
>> I'd like to know the full details why we don't import but write full path
>> though.
>>
>>
>> On 9 Sep 2016 5:28 a.m., "Jakob Odersky" <ja...@odersky.com> wrote:
>>>
>>> +1 to Sean's answer, importing varargs.
>>> In this case the _root_ is also unnecessary (it would be required in
>>> case you were using it in a nested package called "scala" itself)
>>>
>>> On Thu, Sep 8, 2016 at 9:27 AM, Sean Owen <so...@cloudera.com> wrote:
>>> > I think the @_root_ version is redundant because
>>> > @scala.annotation.varargs is redundant. Actually wouldn't we just
>>> > import varargs and write @varargs?
>>> >
>>> > On Thu, Sep 8, 2016 at 1:24 PM, Jacek Laskowski <ja...@japila.pl>
>>> > wrote:
>>> >> Hi,
>>> >>
>>> >> The code is not consistent with @scala.annotation.varargs annotation.
>>> >> There are classes with @scala.annotation.varargs like DataFrameReader
>>> >> or functions as well as examples of @_root_.scala.annotation.varargs,
>>> >> e.g. Window or UserDefinedAggregateFunction.
>>> >>
>>> >> I think it should be consistent and @scala.annotation.varargs only.
>>> >> WDYT?
>>> >>
>>> >> Pozdrawiam,
>>> >> Jacek Laskowski
>>> >> ----
>>> >> https://medium.com/@jaceklaskowski/
>>> >> Mastering Apache Spark 2.0 http://bit.ly/mastering-apache-spark
>>> >> Follow me at https://twitter.com/jaceklaskowski
>>> >>
>>> >> ---------------------------------------------------------------------
>>> >> To unsubscribe e-mail: dev-unsubscr...@spark.apache.org
>>> >>
>>> >
>>> > ---------------------------------------------------------------------
>>> > To unsubscribe e-mail: dev-unsubscr...@spark.apache.org
>>> >
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe e-mail: dev-unsubscr...@spark.apache.org
>>>
>



-- 
Marcelo

---------------------------------------------------------------------
To unsubscribe e-mail: dev-unsubscr...@spark.apache.org

Reply via email to