)]}'
{"trove/common/template.py":[{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"fdce368dd284bf1b9a737817894a51fd85523ece","unresolved":false,"context_lines":[{"line_number":17,"context_line":"_ENV \u003d None"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"def get_env():"},{"line_number":21,"context_line":"    \"\"\" Load jinja environment if it has not already been loaded \"\"\""},{"line_number":22,"context_line":"    global _ENV"},{"line_number":23,"context_line":"    if not _ENV:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FwGY%3D","line":20,"updated":"2013-07-23 21:49:55.000000000","message":"there is no reason to have these functions. We should just have a global variable ENV \u003d jinja2.Environment(...) cut out the middlemen.","commit_id":"ab77c5915c16ec312dc2fd15a71b85360e0f32c9"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"0e557a86eb00da0645721c4df7183dc07e96efe6","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        \"\"\""},{"line_number":52,"context_line":"        self.config_location \u003d self._config_paths[service_type]"},{"line_number":53,"context_line":"        self.flavor_dict \u003d flavor_dict"},{"line_number":54,"context_line":"        template_filename \u003d \"%s.config.template\" % database_type"},{"line_number":55,"context_line":"        self.template \u003d get_env().get_template(template_filename)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def render(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FrvU%3D","line":54,"updated":"2013-07-25 04:47:04.000000000","message":"shouldnt this be service_type?","commit_id":"ab77c5915c16ec312dc2fd15a71b85360e0f32c9"},{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"9b9d61e1afa3adca711d64148ddb62cf031ac833","unresolved":false,"context_lines":[{"line_number":17,"context_line":"_ENV \u003d None"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"def get_env():"},{"line_number":21,"context_line":"    \"\"\" Load jinja environment if it has not already been loaded \"\"\""},{"line_number":22,"context_line":"    global _ENV"},{"line_number":23,"context_line":"    if not _ENV:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FaR8%3D","line":20,"updated":"2013-07-31 16:11:18.000000000","message":"I would like us to cut out the middlemen here. There is no need to lazy load the jinja environment. We can just have a global \u0027ENV\u0027 variable set to the jinja2.Environment(...)","commit_id":"80fa789102b23d7f4acb4c7636cce9c233e19ed6"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"bda5ba74dc729ed49dd69e5ef955c8850b5cb28a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.config_location \u003d self._config_paths[service_type]"},{"line_number":43,"context_line":"        self.flavor_dict \u003d flavor_dict"},{"line_number":44,"context_line":"        template_filename \u003d \"%s.config.template\" % service_type"},{"line_number":45,"context_line":"        self.template \u003d get_env().get_template(template_filename)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def render(self):"},{"line_number":48,"context_line":"        \"\"\" Renders the jinja template"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FaMA%3D","line":45,"updated":"2013-07-31 16:30:55.000000000","message":"Where did get_env go?\nDid you mean create_env...","commit_id":"180420f961cc850ef43a47e1ab0ec8a027daab6f"},{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"a21b0c55ae174a3198a7c508bd8416b2e1413c38","unresolved":false,"context_lines":[{"line_number":20,"context_line":"]))"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def create_env():"},{"line_number":24,"context_line":"    \"\"\" Create the jinja environment \"\"\""},{"line_number":25,"context_line":"    loader \u003d jinja2.ChoiceLoader(["},{"line_number":26,"context_line":"        jinja2.FileSystemLoader(\"/etc/trove/templates\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAQn%2F%2FaAM%3D","line":23,"updated":"2013-07-31 17:07:30.000000000","message":"Nit: this is no longer needed and can be removed","commit_id":"60bf9da5846c7f141308e793a3efc14d5a0ed2d7"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"88b96b696b0f04aa1285a989fdbd08945b6b973b","unresolved":false,"context_lines":[{"line_number":20,"context_line":"]))"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def create_env():"},{"line_number":24,"context_line":"    \"\"\" Create the jinja environment \"\"\""},{"line_number":25,"context_line":"    loader \u003d jinja2.ChoiceLoader(["},{"line_number":26,"context_line":"        jinja2.FileSystemLoader(\"/etc/trove/templates\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAQn%2F%2FaBM%3D","line":23,"updated":"2013-07-31 17:05:31.000000000","message":"Now that we\u0027re creating the env on the fly, do we still need this function? (I don\u0027t see it being called anywhere).","commit_id":"60bf9da5846c7f141308e793a3efc14d5a0ed2d7"}]}