)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"e3c80ce806ff0ee9c1378e3b71aed5c0929d6501","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dcc5d501_30182856","updated":"2022-12-21 06:19:26.000000000","message":"Sorry for my late submit. Please kindly review this spec.","commit_id":"cc86f09f26a0159c3ae9b36168f1c9efcd0ada9e"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ecff37b6_e66aa4c0","updated":"2022-12-26 02:28:37.000000000","message":"Thank you for your review. Please kindly check my replies and updated PS.","commit_id":"35846528d318cd02c61c882624d332f4ba706bab"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"b7092bdd5df99812f44612240d082c1c79dbf836","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"040fe6c0_3f77d24e","updated":"2022-12-26 11:36:03.000000000","message":"Thank you for your comment.","commit_id":"2948d6fa57bc61d0957c5b20ea2187c9bf4702cb"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"70671a724145d3b3e8f86dff1a07e95d0660de77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f4aa0d2e_d6a7e00c","updated":"2022-12-27 07:05:30.000000000","message":"Thank you for your review.","commit_id":"e287ba86b8bba9789728c28c5f32b9f3a76624d6"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"21a6cb54ba38d54c50e0aa62324f774729a5c766","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"63b15991_0e7e93b5","updated":"2022-12-27 16:04:41.000000000","message":"Thanks.\nPlease wait for others\u0027 review.","commit_id":"e287ba86b8bba9789728c28c5f32b9f3a76624d6"}],"specs/2023.1/support-tacker-db-manage-postgresql.rst":[{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  $ /usr/local/bin/tacker-db-manage --config-file \\"},{"line_number":28,"context_line":"  /etc/tacker/tacker.conf upgrade head"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"DevStack [#Dev-stack]_ that inclues a set of scripts and utilities"},{"line_number":31,"context_line":"to develop OpenStack environments also support MySQL and PostgreSQL."},{"line_number":32,"context_line":"However, the current Tacker and ``tacker-db-manage`` does not support"},{"line_number":33,"context_line":"PostgreSQL and cause some practical errors due to differences"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b492a084_3a1d3c4a","line":30,"range":{"start_line":30,"start_character":28,"end_line":30,"end_character":36},"updated":"2022-12-22 18:20:18.000000000","message":"typo: includes","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  $ /usr/local/bin/tacker-db-manage --config-file \\"},{"line_number":28,"context_line":"  /etc/tacker/tacker.conf upgrade head"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"DevStack [#Dev-stack]_ that inclues a set of scripts and utilities"},{"line_number":31,"context_line":"to develop OpenStack environments also support MySQL and PostgreSQL."},{"line_number":32,"context_line":"However, the current Tacker and ``tacker-db-manage`` does not support"},{"line_number":33,"context_line":"PostgreSQL and cause some practical errors due to differences"}],"source_content_type":"text/x-rst","patch_set":2,"id":"d726785b_ff730ccc","line":30,"range":{"start_line":30,"start_character":28,"end_line":30,"end_character":36},"in_reply_to":"b492a084_3a1d3c4a","updated":"2022-12-26 02:28:37.000000000","message":"I fixed it in PS3.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  $ /usr/local/bin/tacker-db-manage --config-file \\"},{"line_number":28,"context_line":"  /etc/tacker/tacker.conf upgrade head"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"DevStack [#Dev-stack]_ that inclues a set of scripts and utilities"},{"line_number":31,"context_line":"to develop OpenStack environments also support MySQL and PostgreSQL."},{"line_number":32,"context_line":"However, the current Tacker and ``tacker-db-manage`` does not support"},{"line_number":33,"context_line":"PostgreSQL and cause some practical errors due to differences"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3bddd6dc_1028fe9e","line":30,"range":{"start_line":30,"start_character":28,"end_line":30,"end_character":36},"in_reply_to":"d726785b_ff730ccc","updated":"2022-12-26 04:30:17.000000000","message":"confirmed","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":64,"context_line":"  * - MEDIUMTEXT could not be used in PostgreSQL."},{"line_number":65,"context_line":"    - Replace TEXT."},{"line_number":66,"context_line":"  * - PostgreSQL ENUM type requires a name."},{"line_number":67,"context_line":"    - Add a name attribute (e.g name\u003d`varbosity`)."},{"line_number":68,"context_line":"  * - Json common functions (e.g, json_extract(), json_length()) could not be used in PostgreSQL."},{"line_number":69,"context_line":"    - Replace decode(), json_array_length() and json search methods."},{"line_number":70,"context_line":"  * - SQL words such as CHANGE and GENERATED could not be used in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"afaabcd4_04e184e5","line":67,"range":{"start_line":67,"start_character":38,"end_line":67,"end_character":47},"updated":"2022-12-22 18:20:18.000000000","message":"verbosity?","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":64,"context_line":"  * - MEDIUMTEXT could not be used in PostgreSQL."},{"line_number":65,"context_line":"    - Replace TEXT."},{"line_number":66,"context_line":"  * - PostgreSQL ENUM type requires a name."},{"line_number":67,"context_line":"    - Add a name attribute (e.g name\u003d`varbosity`)."},{"line_number":68,"context_line":"  * - Json common functions (e.g, json_extract(), json_length()) could not be used in PostgreSQL."},{"line_number":69,"context_line":"    - Replace decode(), json_array_length() and json search methods."},{"line_number":70,"context_line":"  * - SQL words such as CHANGE and GENERATED could not be used in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"af4cfac3_4f91b557","line":67,"range":{"start_line":67,"start_character":38,"end_line":67,"end_character":47},"in_reply_to":"7b3329af_c6f3a3a7","updated":"2022-12-26 04:30:17.000000000","message":"confirmed","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":64,"context_line":"  * - MEDIUMTEXT could not be used in PostgreSQL."},{"line_number":65,"context_line":"    - Replace TEXT."},{"line_number":66,"context_line":"  * - PostgreSQL ENUM type requires a name."},{"line_number":67,"context_line":"    - Add a name attribute (e.g name\u003d`varbosity`)."},{"line_number":68,"context_line":"  * - Json common functions (e.g, json_extract(), json_length()) could not be used in PostgreSQL."},{"line_number":69,"context_line":"    - Replace decode(), json_array_length() and json search methods."},{"line_number":70,"context_line":"  * - SQL words such as CHANGE and GENERATED could not be used in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"7b3329af_c6f3a3a7","line":67,"range":{"start_line":67,"start_character":38,"end_line":67,"end_character":47},"in_reply_to":"afaabcd4_04e184e5","updated":"2022-12-26 02:28:37.000000000","message":"That\u0027s right.\nI fixed it in PS3.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":96,"context_line":"  * - jsonutils.dump_as_bytes() converts json to binary data."},{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bd6bd3fb_8b69260f","line":99,"updated":"2022-12-22 18:20:18.000000000","message":"typo: empty","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":96,"context_line":"  * - jsonutils.dump_as_bytes() converts json to binary data."},{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"78942eff_ea65f773","line":99,"in_reply_to":"7565b821_5fd6bc1c","updated":"2022-12-26 04:30:17.000000000","message":"confirmed","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":96,"context_line":"  * - jsonutils.dump_as_bytes() converts json to binary data."},{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7565b821_5fd6bc1c","line":99,"in_reply_to":"bd6bd3fb_8b69260f","updated":"2022-12-26 02:28:37.000000000","message":"I fixed it in PS3.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b8203072_7b383e43","line":100,"updated":"2022-12-22 18:20:18.000000000","message":"typo: column","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"}],"source_content_type":"text/x-rst","patch_set":2,"id":"c41b601c_446b3757","line":100,"in_reply_to":"90c0cf09_638c14df","updated":"2022-12-26 04:30:17.000000000","message":"confirmed","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":97,"context_line":"    - Use jsonutils.dumps(value)."},{"line_number":98,"context_line":"  * - \u003cclass `dict`\u003e initialization is different (NULL and empty)."},{"line_number":99,"context_line":"    - Support emtpy dict."},{"line_number":100,"context_line":"  * - Type comparison errors in ``deleted`` culumn."},{"line_number":101,"context_line":"    - Fix the type of comparison destinations."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"}],"source_content_type":"text/x-rst","patch_set":2,"id":"90c0cf09_638c14df","line":100,"in_reply_to":"b8203072_7b383e43","updated":"2022-12-26 02:28:37.000000000","message":"I fixed it in PS3.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"95c05315_d4f0dcb5","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"updated":"2022-12-22 18:20:18.000000000","message":"I understood that we need to change, but what are concrete changes due to this need?\nIs this just documentation change? plan to provide sample configuration? or need to change/add some Tacker features?","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"f82f193711efa38d9b313763fd42b1aad928ce5f","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7441fb99_b6353849","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"08e00126_b7b471a9","updated":"2022-12-27 06:19:34.000000000","message":"Thanks, understood.\nIt would be good if that direction is described.\n\n(I think it is \"* Add PostgreSQL install manuals to Tacker Installation Guide.\" in Work Items, but corresponding changes should be included in Proposed Change section.)","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"911b7e3c_bd86b64c","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"4ac06c67_ce8562e5","updated":"2022-12-26 04:30:17.000000000","message":"Thanks.\nIn my opinion, sample code itself is not required, but it would be better to describe that there are several errors and need to add branching process.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"70671a724145d3b3e8f86dff1a07e95d0660de77","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"100de279_5c599bd8","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"7441fb99_b6353849","updated":"2022-12-27 07:05:30.000000000","message":"Thank you for your comments.\nI added the direction as a NOTE and fixed the Proposed Change section in PS5.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"b7092bdd5df99812f44612240d082c1c79dbf836","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"edd95085_7a27c9a3","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"911b7e3c_bd86b64c","updated":"2022-12-26 11:36:03.000000000","message":"Thank you for your opinion.\nI fixed the description to explain how to change like this in PS4.\n```\nThese functions will be\nimplemented by changing parameters or\nusing branching process in tacker-db-manage.\n```","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"4ac06c67_ce8562e5","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"95c05315_d4f0dcb5","updated":"2022-12-26 02:28:37.000000000","message":"Thank you for your confirmation.\nThere are several errors when Tacker and tacker-db-manage initializes PostgreSQL as a db-backend,\nso we will add \"PostgreSQL branching process\" to tacker-db-manage and Tacker like the following example.\n```\n    bind \u003d op.get_bind()\n    engine \u003d bind.engine\n    if engine.name \u003d\u003d \u0027postgresql\u0027:     # for PostgreSQL\n        type \u003d sa.VARCHAR(length\u003d65535)\n    else:\n        type \u003d sa.TEXT(length\u003d65535)    # for MySQL (Current setting)\n```\n\nI think it is better to add the sample codes like this to the spec, what do you think about this idea?","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"01f40b340c817c0ca684ab4fb4f9401bcc1272d4","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"f876015b_8535ca1a","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"edd95085_7a27c9a3","updated":"2022-12-27 05:09:58.000000000","message":"Thanks.\nHow about putting \"by configuration\" in your PS4 changes?\n\nThe point of my comment is to descrbe need of this \"Add configuration...\".\ni.e. the process is branched by configuation. (Is my understanding correct?)","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"415341a243790656b62003cb749c559422e8b82f","unresolved":true,"context_lines":[{"line_number":103,"context_line":"Add configuration to set PostgreSQL as Tacker DB backend"},{"line_number":104,"context_line":"--------------------------------------------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The configuration of PostgreSQL (`postgresql.conf`)"},{"line_number":107,"context_line":"and Tacker (`tacker.conf`) also need to be changed"},{"line_number":108,"context_line":"when Tacker uses PostgreSQL as a DB backend."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"* postgresql.conf"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"08e00126_b7b471a9","line":108,"range":{"start_line":106,"start_character":0,"end_line":108,"end_character":44},"in_reply_to":"f876015b_8535ca1a","updated":"2022-12-27 05:19:57.000000000","message":"Thank you for your comments. I understood.\nWe do not add the configuration settings or samples about this feature.\nInstead, we will add example configurations for PostgreSQL to the Installation Guide.\nUsers can use PostgreSQL as Tacker DB by applying these settings themselves.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"ea8b19018bb60ba08b54cf539c402720c2f04e3d","unresolved":true,"context_lines":[{"line_number":153,"context_line":"    \"tacker-db-manage\" --\u003e \"Tacker\""},{"line_number":154,"context_line":"      [label \u003d \"\"];"},{"line_number":155,"context_line":"    \"Tacker\" --\u003e \"Client\""},{"line_number":156,"context_line":"      [label \u003d \"Succeeed\"];"},{"line_number":157,"context_line":"  }"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"#. The Client creates Tacker DB and user in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"3c4976eb_c4d3aeb1","line":156,"updated":"2022-12-22 18:20:18.000000000","message":"typo: Succeed? Succeeded?","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"8b63f92668b824799b80893d2ab0bd979291fa52","unresolved":true,"context_lines":[{"line_number":153,"context_line":"    \"tacker-db-manage\" --\u003e \"Tacker\""},{"line_number":154,"context_line":"      [label \u003d \"\"];"},{"line_number":155,"context_line":"    \"Tacker\" --\u003e \"Client\""},{"line_number":156,"context_line":"      [label \u003d \"Succeeed\"];"},{"line_number":157,"context_line":"  }"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"#. The Client creates Tacker DB and user in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"66d1d7ce_ec6b16bb","line":156,"in_reply_to":"3c4976eb_c4d3aeb1","updated":"2022-12-26 02:28:37.000000000","message":"\"Succeeded\" is correct, so I fixed it in PS3.","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"03fe59884b3573d09e846229a42e5664b7e5afd5","unresolved":true,"context_lines":[{"line_number":153,"context_line":"    \"tacker-db-manage\" --\u003e \"Tacker\""},{"line_number":154,"context_line":"      [label \u003d \"\"];"},{"line_number":155,"context_line":"    \"Tacker\" --\u003e \"Client\""},{"line_number":156,"context_line":"      [label \u003d \"Succeeed\"];"},{"line_number":157,"context_line":"  }"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"#. The Client creates Tacker DB and user in PostgreSQL."}],"source_content_type":"text/x-rst","patch_set":2,"id":"82ae39e0_489a016c","line":156,"in_reply_to":"66d1d7ce_ec6b16bb","updated":"2022-12-26 04:30:17.000000000","message":"confirmed","commit_id":"aa3677ac3dda1d2b10aa35aa80c072d67bb79288"}]}