)]}'
{"specs/kilo/edp-add-hbase-sharelib.rst":[{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"3e4816e6c6cf51d8877a63241a6d1e640829a0a7","unresolved":false,"context_lines":[{"line_number":105,"context_line":"Note that if for some reason an advanced user has already set `oozie.libpath`"},{"line_number":106,"context_line":"to a value, the location of the HBase lib should be added to the value (which"},{"line_number":107,"context_line":"may be a comma-separated list)."},{"line_number":108,"context_line":"  "},{"line_number":109,"context_line":"  "},{"line_number":110,"context_line":"Alternatives"},{"line_number":111,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"da86d52c_502e7d8d","line":108,"updated":"2015-02-16 13:31:25.000000000","message":"Please, remove trailing spaces here and on other lines.","commit_id":"d723be5352e096552cf1894008444a94b8ddbb24"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"8a59ae2c5eba04f4065818a0f52889de968d7bc4","unresolved":false,"context_lines":[{"line_number":105,"context_line":"Note that if for some reason an advanced user has already set `oozie.libpath`"},{"line_number":106,"context_line":"to a value, the location of the HBase lib should be added to the value (which"},{"line_number":107,"context_line":"may be a comma-separated list)."},{"line_number":108,"context_line":"  "},{"line_number":109,"context_line":"  "},{"line_number":110,"context_line":"Alternatives"},{"line_number":111,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"da86d52c_c97ba479","line":108,"in_reply_to":"da86d52c_502e7d8d","updated":"2015-02-16 14:50:20.000000000","message":"done","commit_id":"d723be5352e096552cf1894008444a94b8ddbb24"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"3e4816e6c6cf51d8877a63241a6d1e640829a0a7","unresolved":false,"context_lines":[{"line_number":173,"context_line":"Assignee(s)"},{"line_number":174,"context_line":"-----------"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"Primary assignee:"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Other contributors:"},{"line_number":179,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da86d52c_f03851d6","line":176,"updated":"2015-02-16 13:31:25.000000000","message":"Should be added at least one primary assignee.","commit_id":"d723be5352e096552cf1894008444a94b8ddbb24"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"8a59ae2c5eba04f4065818a0f52889de968d7bc4","unresolved":false,"context_lines":[{"line_number":173,"context_line":"Assignee(s)"},{"line_number":174,"context_line":"-----------"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"Primary assignee:"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Other contributors:"},{"line_number":179,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da86d52c_49e6d445","line":176,"in_reply_to":"da86d52c_f03851d6","updated":"2015-02-16 14:50:20.000000000","message":"done, intentional there is no primary, I added myself as \"other contributor\" but I probably am not the primary for this.  One of the developers working on CDH plugin enhancements should take it. I just figured out the solution on their behalf.","commit_id":"d723be5352e096552cf1894008444a94b8ddbb24"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"f68b0b8d5f97c15df4f23a214ec7cec9bc5cb7b8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"an extra classpath value. Instead, the Oozie solution to this problem is"},{"line_number":33,"context_line":"to create a directory of JAR files in hdfs and then set the"},{"line_number":34,"context_line":"`oozie.libpath` configuration property on the job to that location."},{"line_number":35,"context_line":"This causes Oozie to make all of the jars in the directory available to the job."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Sahara currently supports setting the `oozie.libpath` configuration on a job"},{"line_number":38,"context_line":"but there is no existing collection of HBase JARs to reference. Users can log into"}],"source_content_type":"text/x-rst","patch_set":3,"id":"da86d52c_5ef03e66","line":35,"updated":"2015-02-16 20:19:32.000000000","message":"this line is causing errors for going over 79 chars","commit_id":"09dc9e4be6a5aab9ee193bcf95f333207993834b"},{"author":{"_account_id":13953,"name":"Elise Gafford","email":"egafford@redhat.com","username":"egafford"},"change_message_id":"03b3072dea9683ec5224a3bbcf3c27823f3b8463","unresolved":false,"context_lines":[{"line_number":99,"context_line":"  leaked into Sahara\u0027s interfaces already but there is no reason to"},{"line_number":100,"context_line":"  add to it."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Alternatively, we can use an `edp.use_hbase_lib` boolean configuration"},{"line_number":103,"context_line":"parameter to specify whether a job should use the HBase hdfs library. If this"},{"line_number":104,"context_line":"configuration parameter is True, EDP can retrieve the hdfs location"},{"line_number":105,"context_line":"from the utility class described above and set `oozie.libpath` accordingly."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_ff7d476e","line":102,"updated":"2015-02-17 19:39:12.000000000","message":"It looks below as though this option is the implementation you\u0027re suggesting, but the \"alternatively\" here makes it a bit unclear which implementation you\u0027re suggesting (in this paragraph).","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"fc8879ee292fbdad7930cbe55c746cb88f6db1bf","unresolved":false,"context_lines":[{"line_number":99,"context_line":"  leaked into Sahara\u0027s interfaces already but there is no reason to"},{"line_number":100,"context_line":"  add to it."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Alternatively, we can use an `edp.use_hbase_lib` boolean configuration"},{"line_number":103,"context_line":"parameter to specify whether a job should use the HBase hdfs library. If this"},{"line_number":104,"context_line":"configuration parameter is True, EDP can retrieve the hdfs location"},{"line_number":105,"context_line":"from the utility class described above and set `oozie.libpath` accordingly."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_483e97b9","line":102,"in_reply_to":"da86d52c_ff7d476e","updated":"2015-02-23 20:23:12.000000000","message":"Done","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"a64238b18e6f67a7f87580fbf8726f862d175cf5","unresolved":false,"context_lines":[{"line_number":99,"context_line":"  leaked into Sahara\u0027s interfaces already but there is no reason to"},{"line_number":100,"context_line":"  add to it."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Alternatively, we can use an `edp.use_hbase_lib` boolean configuration"},{"line_number":103,"context_line":"parameter to specify whether a job should use the HBase hdfs library. If this"},{"line_number":104,"context_line":"configuration parameter is True, EDP can retrieve the hdfs location"},{"line_number":105,"context_line":"from the utility class described above and set `oozie.libpath` accordingly."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_a8e9d31b","line":102,"in_reply_to":"da86d52c_ff7d476e","updated":"2015-02-17 20:44:12.000000000","message":"agreed, maybe use different phrasing if the \"alternative\" option here is just an implementation detail. otherwise, if it truly is an alternative, then move it to the next section.","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":13953,"name":"Elise Gafford","email":"egafford@redhat.com","username":"egafford"},"change_message_id":"03b3072dea9683ec5224a3bbcf3c27823f3b8463","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"We may want a simple checkbox option on the UI for Java actions"},{"line_number":167,"context_line":"to set the `edp.use_hbase_libpath` config so that users don\u0027t need to"},{"line_number":168,"context_line":"add it by hand."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_5f19f378","line":168,"updated":"2015-02-17 19:39:12.000000000","message":"Would this be included in the full configuration or hidden by default?","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"557ace029c0e303f7bed100ae9c7f035821b9cc9","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"We may want a simple checkbox option on the UI for Java actions"},{"line_number":167,"context_line":"to set the `edp.use_hbase_libpath` config so that users don\u0027t need to"},{"line_number":168,"context_line":"add it by hand."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba7be1f8_7527d612","line":168,"in_reply_to":"ba7be1f8_9a8da034","updated":"2015-02-23 20:24:28.000000000","message":"oops, old draft, disregard","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"fc8879ee292fbdad7930cbe55c746cb88f6db1bf","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"We may want a simple checkbox option on the UI for Java actions"},{"line_number":167,"context_line":"to set the `edp.use_hbase_libpath` config so that users don\u0027t need to"},{"line_number":168,"context_line":"add it by hand."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba7be1f8_9a8da034","line":168,"in_reply_to":"da86d52c_5f19f378","updated":"2015-02-23 20:23:12.000000000","message":"Hidden would be great, if there is a place to hide it.  Not sure what the new forms will look like post-wizardry.  croberts?","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"},{"author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"change_message_id":"27544fa901fb9ffe7da1b8bc8c83d0fc967c729f","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"We may want a simple checkbox option on the UI for Java actions"},{"line_number":167,"context_line":"to set the `edp.use_hbase_libpath` config so that users don\u0027t need to"},{"line_number":168,"context_line":"add it by hand."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba7be1f8_7a12acb5","line":168,"in_reply_to":"da86d52c_5f19f378","updated":"2015-02-20 14:44:13.000000000","message":"I suspect it would be a checkbox that would always show up on the configs page (but only when the job type is appropriate).  Always open to better ideas though.","commit_id":"8fea7b7db9f32d2b8d2ef3e0f0f16874f2745377"}]}
