Merge lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only into lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/trunk

Proposed by Cory Johns on 2015-08-13
Status: Merged
Merged at revision: 17
Proposed branch: lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only
Merge into: lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
hooks/common.py (+1/-1)
To merge this branch: bzr merge lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only
Reviewer Review Type Date Requested Status
Kevin W Monroe 2015-08-13 Approve on 2015-08-18
Review via email: mp+268002@code.launchpad.net

Description of the change

Added support for HDFS-only, for use with the new hadoop-plugin

To post a comment you must log in.
Kevin W Monroe (kwmonroe) wrote :

LGTM

review: Approve
17. By Kevin W Monroe on 2015-08-18

[merge] Flume only really requires HDFS

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/common.py'
2--- hooks/common.py 2015-07-28 03:28:46 +0000
3+++ hooks/common.py 2015-08-13 21:29:09 +0000
4@@ -62,7 +62,7 @@
5 ],
6 'requires': [
7 flume.verify_resources,
8- jujubigdata.relations.HadoopPlugin(),
9+ jujubigdata.relations.HadoopPlugin(hdfs_only=True),
10 callbacks.update_blocked_status, # not really a requirement, but best way to fit into framework
11 ],
12 'callbacks': [

Subscribers

People subscribed via source and target branches

to all changes: