Github user kwmonroe commented on a diff in the pull request:
    --- Diff: bigtop-packages/src/charm/hive/layer-hive/reactive/ ---
    @@ -78,9 +87,13 @@ def install_hive(hadoop):
             hbserver = None
    +    # Get zookeepers or None
    +    zks = KV.get('zookeepers', None)
    --- End diff --
    I'm curious why we're using `unitdata.kv` here.  If you look a few lines 
higher at the hbase block, we could use the same logic to get zks when 
`is_state(zookeeper.ready)` is available.


Reply via email to