spark-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hyukjin Kwon <gurwls...@gmail.com>
Subject Re: Component naming in the PR title
Date Sun, 13 Nov 2016 22:02:50 GMT
I see. I was just curious as I find myself hesitating when I open a PR time
to time.

Thank you both for echoing!

On 14 Nov 2016 5:02 a.m., "Sean Owen" <sowen@cloudera.com> wrote:

> Yes they really correspond to, if anything, the categories at
> spark-prs.appspot.com . They aren't that consistently used however and
> there isn't really a definite list. It is really mostly of use for the fact
> that it tags emails in a way people can filter semi-effectively. So I think
> we have left it at an informal mechanism rather than make yet another list
> of categories to maintain.
>
> On Sat, Nov 12, 2016, 18:27 Hyukjin Kwon <gurwls223@gmail.com> wrote:
>
>> Hi all,
>>
>>
>> First of all, this might be minor but I just have been curious of
>> different PR titles in particular component part. So, I looked through
>> Spark wiki again and I found the description not quite the same with the
>> PRs.
>>
>> It seems, it is said,
>> Pull Request
>>
>> ...
>>
>>    1.
>>
>>       The PR title should be of the form [SPARK-xxxx] [COMPONENT] Title,
>>       where SPARK-xxxx is the relevant JIRA number, COMPONENT is one of the PR
>>       categories shown at https://spark-prs.appspot.com/ and Title may
>>       be the JIRA's title or a more specific title describing the PR itself.
>>
>>
>> ...
>>
>> So, It seems the component should be one of
>>
>> SQL, MLlib, Core, Python, Scheduler, Build, Docs, Streaming, Mesos, Web
>> UI, YARN, GraphX, R.
>>
>> If the component refers the ones specified in the JIRA, then, it seems it
>> should be one of
>>
>> Block Manager, Build, Deploy, Documentation, DStream, EC2, Examples,
>> GraphX, Input/Output, Java API, Mesos, ML, MLlib, Optimizer, Project Infra,
>> PySpark, Scheduler, Shuffle, Spark Core, Spark Shell, Spark Submit, SparkR,
>> SQL, Structured Streaming, Tests, Web UI, Windows, YARN
>>
>> It seems they are a bit different with the PRs. I hope this is clarified
>> in more details (including whether it should be all upper-cased or just the
>> same with the component name maybe).
>>
>>
>> Thanks.
>>
>

Mime
View raw message