Uh oh!
There was an error while loading. Please reload this page.
[SPARK-20400][DOCS] Remove References to 3rd Party Vendor Tools - #17695
[SPARK-20400][DOCS] Remove References to 3rd Party Vendor Tools#17695bllchmbrs wants to merge 2 commits into
Conversation
bllchmbrs
commented
Apr 20, 2017
This should be on hold until a JIRA resolution, I'd like to hear what others say. |
gatorsmile
commented
Apr 20, 2017
LGTM |
SparkQA
commented
Apr 20, 2017
Test build #75969 has finished for PR 17695 at commit
|
| The location of these configuration files varies across CDH and HDP versions, but | ||
| a common location is inside of `/etc/hadoop/conf`. Some tools, such as Cloudera Manager, create | ||
| The location of these configuration files varies across hadoop versions, but |
| The location of these configuration files varies across CDH and HDP versions, but | ||
| a common location is inside of `/etc/hadoop/conf`. Some tools, such as Cloudera Manager, create | ||
| The location of these configuration files varies across hadoop versions, but | ||
| a common location is inside of `/etc/hadoop/conf`. Some tools create |
There was a problem hiding this comment.
This also won't matter either way, but is an example of an instance where naming a specific product is helpful information for users. There's no policy against this. But, it can be removed, sure, it's trivial either way.
bllchmbrs
commented
Apr 23, 2017
Thanks for the info @srowen - this should be better now. |
SparkQA
commented
Apr 23, 2017
Test build #76086 has finished for PR 17695 at commit
|
| The location of these configuration files varies across CDH and HDP versions, but | ||
| a common location is inside of `/etc/hadoop/conf`. Some tools, such as Cloudera Manager, create | ||
| The location of these configuration files varies across Hadoop versions, but |
There was a problem hiding this comment.
Hm, I guess one issue I've realized now is that it doesn't really vary across versions of Hadoop but could vary according to packaging and distribution. If you change it again, also consider fixing the existing "mechanisms" typo in line 2253
srowen
commented
Apr 26, 2017
OK, merged it anyway to master/2.2 |
## What changes were proposed in this pull request? Simple documentation change to remove explicit vendor references. ## How was this patch tested? NA Please review http://spark.apache.org/contributing.html before opening a pull request. Author: anabranch <bill@databricks.com> Closes#17695 from anabranch/remove-vendor. (cherry picked from commit 7a36525) Signed-off-by: Sean Owen <sowen@cloudera.com>
What changes were proposed in this pull request?
Simple documentation change to remove explicit vendor references.
How was this patch tested?
NA
Please review http://spark.apache.org/contributing.html before opening a pull request.