spark-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tom Graves <>
Subject Re: [DISCUSS] Minimize use of MINOR, BUILD, and HOTFIX w/ no JIRA
Date Thu, 07 Jul 2016 18:47:32 GMT
Popping this back up to the dev list again.  I see a bunch of checkins with minor or hotfix.
It seems to me we shouldn't be doing this, but I would like to hear thoughts from others.
 I see no reason we can't have a jira for each of those issues, it only takes a few seconds
to file one and it makes things much easier to track.
For instance, I tend to watch the jiras on the mailing list and if I hit an issue I search
jira to see if there is existing one for it, but if there isn't jira then I don't see and
can't find what someone perhaps already fixed with a [MINOR] checkin.

    On Saturday, June 6, 2015 11:02 AM, Patrick Wendell <> wrote:

 Hey All,

Just a request here - it would be great if people could create JIRA's
for any and all merged pull requests. The reason is that when patches
get reverted due to build breaks or other issues, it is very difficult
to keep track of what is going on if there is no JIRA. Here is a list
of 5 patches we had to revert recently that didn't include a JIRA:

    Revert "[MINOR] [BUILD] Use custom temp directory during build."
    Revert "[SQL] [TEST] [MINOR] Uses a temporary in
HiveThriftServer2Test to ensure expected logging behavior"
    Revert "[BUILD] Always run SQL tests in master build."
    Revert "[MINOR] [CORE] Warn users who try to cache RDDs with
dynamic allocation on."
    Revert "[HOT FIX] [YARN] Check whether `/lib` exists before
listing its files"

The cost overhead of creating a JIRA relative to other aspects of
development is very small. If it's *really* a documentation change or
something small, that's okay.

But anything affecting the build, packaging, etc. These all need to
have a JIRA to ensure that follow-up can be well communicated to all
Spark developers.

Hopefully this is something everyone can get behind, but opened a
discussion here in case others feel differently.

- Patrick

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message