)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7da3e34bfab30a3b27895e8a7f256d135d3a8e0a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"to support retrieving server data in the previous page and provide"},{"line_number":11,"context_line":"previous page link on the response."},{"line_number":12,"context_line":"https://review.openstack.org/#/c/164169/"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id84f226ddcdee01bc2cb023c7370bddf913fa543"},{"line_number":15,"context_line":"Implements: blueprint nova-pagination-prevpage"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3ab451fb_7dd34c39","line":13,"updated":"2015-04-17 18:19:07.000000000","message":"Please also add this line to the commit:\nAPIImpact","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"}],"specs/liberty/approved/nova-pagination-prevpage.rst":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4cb75901a2dbf17ab62d8e911afb682d265c45a6","unresolved":false,"context_lines":[{"line_number":14,"context_line":"However, we may want to retrieve the servers in the previous page. To enable"},{"line_number":15,"context_line":"backward query, the previous page link should be provided for the /servers and"},{"line_number":16,"context_line":"/servers/detail APIs."},{"line_number":17,"context_line":"For more detail, please see Reference section."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Problem description"},{"line_number":20,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ab451fb_6a1dc472","line":17,"updated":"2015-04-17 18:16:14.000000000","message":"This spec must not need external references to make sense, please add all details into the spec.\n\nHonestly, looks like you have most of the details here though, I would just drop this extra note.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"915c707832dd5eb1105374adb564c8a3202abd13","unresolved":false,"context_lines":[{"line_number":44,"context_line":"once on the request:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"* is_reverse: determine whether to retrieve the server data in the previous"},{"line_number":47,"context_line":"              page (\u0027True\u0027 or \u0027False\u0027)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"The caller could specify \u0027is_reverse\u0027 to retrieve server data before the"},{"line_number":50,"context_line":"server specified by the \u0027marker\u0027. By default, is_reverse is False."}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_98252cc4","line":47,"updated":"2015-04-20 01:45:09.000000000","message":"I knew Neutron use \u0027page_reverse\u0027, how about we use the consistent naming?","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":14976,"name":"wuhao","email":"wuhao@unitedstack.com","username":"wuhao"},"change_message_id":"de97ab358e0897947ab20a9027b302b10451e3fe","unresolved":false,"context_lines":[{"line_number":44,"context_line":"once on the request:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"* is_reverse: determine whether to retrieve the server data in the previous"},{"line_number":47,"context_line":"              page (\u0027True\u0027 or \u0027False\u0027)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"The caller could specify \u0027is_reverse\u0027 to retrieve server data before the"},{"line_number":50,"context_line":"server specified by the \u0027marker\u0027. By default, is_reverse is False."}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_4c7472fc","line":47,"in_reply_to":"dac4157f_98252cc4","updated":"2015-04-20 06:39:45.000000000","message":"Thank you for reminding me of this. Actually, I found your patch in neutron. I\u0027ll use \u0027page_reverse\u0027 in the next patch.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4cb75901a2dbf17ab62d8e911afb682d265c45a6","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"If we don\u0027t add another \u0027is_reverse\u0027 param, we could only provide the previous"},{"line_number":65,"context_line":"page link with reverse sort direction. However, the result will be in an"},{"line_number":66,"context_line":"opposite order."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Data model impact"},{"line_number":69,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ab451fb_ddc14082","line":66,"updated":"2015-04-17 18:16:14.000000000","message":"I have a slight preference for the next and previous page to be dealt with in the same way, so if one is a link in the doc, lets make them both a link in the doc.\n\nCan we link to examples from other projects to describe with approach would be consistent.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":14976,"name":"wuhao","email":"wuhao@unitedstack.com","username":"wuhao"},"change_message_id":"ac44be78e9515e0f3a4f26f4ca5297f516d6ca8a","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"If we don\u0027t add another \u0027is_reverse\u0027 param, we could only provide the previous"},{"line_number":65,"context_line":"page link with reverse sort direction. However, the result will be in an"},{"line_number":66,"context_line":"opposite order."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Data model impact"},{"line_number":69,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_8c096a34","line":66,"in_reply_to":"3ab451fb_ddc14082","updated":"2015-04-20 06:49:20.000000000","message":"Yeah, it\u0027ll be more friendly to have a same link. But there isn\u0027t a good solution to find the first item in the previous link. A similar approach was implemented in Neutron. https://review.openstack.org/#/c/15384/.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"915c707832dd5eb1105374adb564c8a3202abd13","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"A request example will be:"},{"line_number":81,"context_line":"* http://cloudcontroller:8774/v2.1/{tenant_id}/servers?is_reverse\u003dTrue"},{"line_number":82,"context_line":"* http://cloudcontroller:8774/v2.1/{tenant_id}/servers/detail?is_reverse\u003dTrue"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The previous link will be added to the response"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_f81c2879","line":82,"updated":"2015-04-20 01:45:09.000000000","message":"What is expect response with is_reverse and without marker?","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"915c707832dd5eb1105374adb564c8a3202abd13","unresolved":false,"context_lines":[{"line_number":91,"context_line":"            {"},{"line_number":92,"context_line":"                \"href\": \"http://cloudcontroller:8774/v2.1/{tenant_id}/servers?\""},{"line_number":93,"context_line":"                        \"is_reverse\u003dTrue\u0026marker\u003d\u003cuuid\u003e\","},{"line_number":94,"context_line":"                \"rel\": \"prev\""},{"line_number":95,"context_line":"            }"},{"line_number":96,"context_line":"        ],"},{"line_number":97,"context_line":"        ..."}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_d811645f","line":94,"updated":"2015-04-20 01:45:09.000000000","message":"Better to describe what the expect response for the last page and first page.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4cb75901a2dbf17ab62d8e911afb682d265c45a6","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            }"},{"line_number":96,"context_line":"        ],"},{"line_number":97,"context_line":"        ..."},{"line_number":98,"context_line":"    }"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"Security impact"},{"line_number":101,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3ab451fb_bd90b496","line":98,"updated":"2015-04-17 18:16:14.000000000","message":"I think the spec template requires json schema addtions to validate the extra parameters.","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"40f3340b0b898c7a05fd6b3861e2f2fa6379dc27","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Performance Impact"},{"line_number":116,"context_line":"------------------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"None"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Other deployer impact"}],"source_content_type":"text/x-rst","patch_set":7,"id":"9aa53dc9_55eb83b0","line":117,"updated":"2015-04-08 15:20:35.000000000","message":"performance should be better, right?","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"40f3340b0b898c7a05fd6b3861e2f2fa6379dc27","unresolved":false,"context_lines":[{"line_number":157,"context_line":"Testing"},{"line_number":158,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"Both unit and Tempest tests need to be added for /servers and /servers/detail"},{"line_number":161,"context_line":"API to test if server data in the previous page is returned when \u0027is_reverse\u0027"},{"line_number":162,"context_line":"is True for various query parameters."},{"line_number":163,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"9aa53dc9_35411faa","line":160,"updated":"2015-04-08 15:20:35.000000000","message":"Guess functional test should cover major function test","commit_id":"509750b398c9ec7b6a1e0ba48400bd94674bc2a3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9c49bb6cf6d047f4b7f6b7baa35ee44cd3a25d71","unresolved":false,"context_lines":[{"line_number":65,"context_line":"If we don\u0027t add another \u0027page_reverse\u0027 param, we could only provide the"},{"line_number":66,"context_line":"previous page link with reverse sort direction. However, the result will be in"},{"line_number":67,"context_line":"an opposite order. Actually, neutron also introduces \u0027page_reverse\u0027 to enable"},{"line_number":68,"context_line":"different page directions."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Data model impact"},{"line_number":71,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_b942320d","line":68,"updated":"2015-06-11 10:34:29.000000000","message":"Yeah, thats a good point.\n\nI just wonder if we can\u0027t just use the marker for the reverse links?","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"aa8653f25571343046d3ee176b1c60163901220e","unresolved":false,"context_lines":[{"line_number":65,"context_line":"If we don\u0027t add another \u0027page_reverse\u0027 param, we could only provide the"},{"line_number":66,"context_line":"previous page link with reverse sort direction. However, the result will be in"},{"line_number":67,"context_line":"an opposite order. Actually, neutron also introduces \u0027page_reverse\u0027 to enable"},{"line_number":68,"context_line":"different page directions."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Data model impact"},{"line_number":71,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_2a5d9be9","line":68,"in_reply_to":"3afb71cf_6bcdf25c","updated":"2015-06-12 15:54:29.000000000","message":"Thats a good point, its worth adding that into the alternatives section I think?","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":14976,"name":"wuhao","email":"wuhao@unitedstack.com","username":"wuhao"},"change_message_id":"0f0a36c9da2553ec1caa565acb66140c90a2e31b","unresolved":false,"context_lines":[{"line_number":65,"context_line":"If we don\u0027t add another \u0027page_reverse\u0027 param, we could only provide the"},{"line_number":66,"context_line":"previous page link with reverse sort direction. However, the result will be in"},{"line_number":67,"context_line":"an opposite order. Actually, neutron also introduces \u0027page_reverse\u0027 to enable"},{"line_number":68,"context_line":"different page directions."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Data model impact"},{"line_number":71,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_6bcdf25c","line":68,"in_reply_to":"3afb71cf_b942320d","updated":"2015-06-11 18:21:11.000000000","message":"I\u0027m open to this. But it seems a little complex. To generate the url of the previous page, you need to find the previous item of the previous page to get the marker. And if the previous page don\u0027t have enough items, the final result of previous link will include some items of current page.","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9c49bb6cf6d047f4b7f6b7baa35ee44cd3a25d71","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        \"servers_links\": ["},{"line_number":94,"context_line":"            {"},{"line_number":95,"context_line":"                \"href\": \"http://cloudcontroller:8774/v2.1/{tenant_id}/servers?\""},{"line_number":96,"context_line":"                        \"page_reverse\u003dFalse\u0026marker\u003d\u003clast_item_uuid\u003e\""},{"line_number":97,"context_line":"                \"rel\": \"next\""},{"line_number":98,"context_line":"            },"},{"line_number":99,"context_line":"            {"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_c73ceae2","line":96,"updated":"2015-06-11 10:34:29.000000000","message":"we default to page_reverse\u003dFalse, so I guess this should stay as:\nmarker\u003d\u003clast_item_uuid\u003e","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":14976,"name":"wuhao","email":"wuhao@unitedstack.com","username":"wuhao"},"change_message_id":"0f0a36c9da2553ec1caa565acb66140c90a2e31b","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        \"servers_links\": ["},{"line_number":94,"context_line":"            {"},{"line_number":95,"context_line":"                \"href\": \"http://cloudcontroller:8774/v2.1/{tenant_id}/servers?\""},{"line_number":96,"context_line":"                        \"page_reverse\u003dFalse\u0026marker\u003d\u003clast_item_uuid\u003e\""},{"line_number":97,"context_line":"                \"rel\": \"next\""},{"line_number":98,"context_line":"            },"},{"line_number":99,"context_line":"            {"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_ab388a3e","line":96,"in_reply_to":"3afb71cf_c73ceae2","updated":"2015-06-11 18:21:11.000000000","message":"Yeah, I\u0027ll change this.","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9c49bb6cf6d047f4b7f6b7baa35ee44cd3a25d71","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            },"},{"line_number":99,"context_line":"            {"},{"line_number":100,"context_line":"                \"href\": \"http://cloudcontroller:8774/v2.1/{tenant_id}/servers?\""},{"line_number":101,"context_line":"                        \"page_reverse\u003dTrue\u0026marker\u003d\u003cfirst_item_uuid\u003e\","},{"line_number":102,"context_line":"                \"rel\": \"previous\""},{"line_number":103,"context_line":"            }"},{"line_number":104,"context_line":"        ],"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_a728ce9a","line":101,"updated":"2015-06-11 10:34:29.000000000","message":"Can\u0027t this just be:\nmarker\u003d\u003citem_just_before_first_item_uuid\u003e","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":14976,"name":"wuhao","email":"wuhao@unitedstack.com","username":"wuhao"},"change_message_id":"0f0a36c9da2553ec1caa565acb66140c90a2e31b","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            },"},{"line_number":99,"context_line":"            {"},{"line_number":100,"context_line":"                \"href\": \"http://cloudcontroller:8774/v2.1/{tenant_id}/servers?\""},{"line_number":101,"context_line":"                        \"page_reverse\u003dTrue\u0026marker\u003d\u003cfirst_item_uuid\u003e\","},{"line_number":102,"context_line":"                \"rel\": \"previous\""},{"line_number":103,"context_line":"            }"},{"line_number":104,"context_line":"        ],"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_ce24f461","line":101,"in_reply_to":"3afb71cf_a728ce9a","updated":"2015-06-11 18:21:11.000000000","message":"we got an item, and use the marker to find the previous page. That sounds more friendly.","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9c49bb6cf6d047f4b7f6b7baa35ee44cd3a25d71","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            }"},{"line_number":140,"context_line":"        ],"},{"line_number":141,"context_line":"        ..."},{"line_number":142,"context_line":"    }"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"Security impact"},{"line_number":145,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3afb71cf_4700fa01","line":142,"updated":"2015-06-11 10:34:29.000000000","message":"did you copy this API from somewhere? like another project that already does this? Sounds like that was neutron, could you please add that detail here?\n\nI think we should try and have the direction proposed or agreed by the API-WG before we merge this:\nhttp://specs.openstack.org/openstack/api-wg/guidelines/pagination_filter_sort.html","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"aa8653f25571343046d3ee176b1c60163901220e","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            }"},{"line_number":140,"context_line":"        ],"},{"line_number":141,"context_line":"        ..."},{"line_number":142,"context_line":"    }"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"Security impact"},{"line_number":145,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_4a269769","line":142,"in_reply_to":"3afb71cf_4700fa01","updated":"2015-06-12 15:54:29.000000000","message":"It is worth saying if another project (neutron?) does something like this already? that would make this easier to merge","commit_id":"8c27bf9318bc04b0149e557016009b3f6c80b86f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"51d189bc5e5bd1f70a6580ede014301eb304f90b","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        ],"},{"line_number":127,"context_line":"        ..."},{"line_number":128,"context_line":"    }"},{"line_number":129,"context_line":"                                                                {"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"The response of the last page:"},{"line_number":132,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa32b979_4e9c265d","line":129,"updated":"2015-06-23 10:19:05.000000000","message":"Nit: not sure what this thing is doing here.","commit_id":"7659061be2134e041a9c99cda61bdea0937fff59"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"51d189bc5e5bd1f70a6580ede014301eb304f90b","unresolved":false,"context_lines":[{"line_number":226,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"[1] http://specs.openstack.org/openstack/neutron-specs/specs/api/"},{"line_number":229,"context_line":"networking_general_api_information.html#pagination"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa32b979_cea8367a","line":229,"updated":"2015-06-23 10:19:05.000000000","message":"Thank you for adding that","commit_id":"7659061be2134e041a9c99cda61bdea0937fff59"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"284a9aedb3135613fe1b17a0a774992d1980acca","unresolved":false,"context_lines":[{"line_number":66,"context_line":"page link with a reverse sort direction. However, the result will be in an"},{"line_number":67,"context_line":"opposite order. And we could also generate the url of the previous page by"},{"line_number":68,"context_line":"using the marker of the previous item of the previous page. That\u0027s a little"},{"line_number":69,"context_line":"complex. If the preivous page don\u0027t have enough items, the final result of the"},{"line_number":70,"context_line":"previous link will include some items of current page."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Data model impact"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da20952f_5a700c72","line":69,"updated":"2015-09-06 05:06:29.000000000","message":"s/preivous/previous/","commit_id":"e280d22f890512945b880d447e94574cfbec7eb3"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"284a9aedb3135613fe1b17a0a774992d1980acca","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"The \u0027page_reverse\u0027 is optional, and it\u0027s \u0027False\u0027 by default. if \u0027marker\u0027 is"},{"line_number":111,"context_line":"not set and \u0027page_reverse\u0027 is set to \u0027False\u0027, the response will be the first"},{"line_number":112,"context_line":"page. And if \u0027page_reverse\u0027 is set to \u0027True\u0027, the repsonse will be the last"},{"line_number":113,"context_line":"page."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"The response of the first page:"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da20952f_fa6a4080","line":112,"updated":"2015-09-06 05:06:29.000000000","message":"s/repsonse/response/","commit_id":"e280d22f890512945b880d447e94574cfbec7eb3"}],"specs/liberty/nova-pagination-prevpage.rst":[{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"f618fc2885c2f14f000226ef4afbe8d741aadb6b","unresolved":false,"context_lines":[{"line_number":46,"context_line":"* is_reverse: determine whether to retrieve the server data in the previous"},{"line_number":47,"context_line":"              page (\u0027True\u0027 or \u0027False\u0027)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"The caller could specify \u0027is_reverse\u0027 to retrieve server data before the "},{"line_number":50,"context_line":"server specified by the \u0027marker\u0027. By default, is_reverse is False."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"For example: /servers?is_reverse\u003d1\u0026marker\u003d\u003cuuid\u003e"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a80dd14_c9ed7fb9","line":49,"updated":"2015-03-17 15:21:50.000000000","message":"can you please run tox before submit to gerrit?","commit_id":"62bbd5a72c188bfaa96ab1f0a2bf940cdbcff3dc"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"8b37979a08d6a1145a6df51e3f8335f6aa60e914","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"* /v3/servers"},{"line_number":84,"context_line":"* /v3/servers/details"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The existing API documentation needs to be updated to include the following"},{"line_number":87,"context_line":"parameter:"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_74c35a8a","line":85,"updated":"2015-03-18 11:48:29.000000000","message":"no v3 now.","commit_id":"62739a00c2b0b4f0b54ba374dde333f66ca5aecc"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"8b37979a08d6a1145a6df51e3f8335f6aa60e914","unresolved":false,"context_lines":[{"line_number":141,"context_line":"* Reverse the sort direction if \u0027is_reverse\u0027 is True"},{"line_number":142,"context_line":"* Reverse the result of server data"},{"line_number":143,"context_line":"* provide the previous page link in \u0027servers_links\u0027 on the response (change"},{"line_number":144,"context_line":"  openstack/compute/views/servers.py, api/openstack/common.py)"},{"line_number":145,"context_line":""},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"Dependecies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_946c4e9f","line":144,"updated":"2015-03-18 11:48:29.000000000","message":"I don\u0027t think you need to document so detail.\nyou can read some approved examples.","commit_id":"62739a00c2b0b4f0b54ba374dde333f66ca5aecc"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"8b37979a08d6a1145a6df51e3f8335f6aa60e914","unresolved":false,"context_lines":[{"line_number":154,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Add new test cases to /servers and /servers/detail API to test if server data"},{"line_number":157,"context_line":"in the previous page is returned when \u0027is_reverse\u0027 is True."},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Documentation Impact"},{"line_number":160,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_f42bca30","line":157,"updated":"2015-03-18 11:48:29.000000000","message":"unit test case functional test case and tempest test cases should be coverd","commit_id":"62739a00c2b0b4f0b54ba374dde333f66ca5aecc"}]}