micahstubbs opened a new pull request, #2222: URL: https://github.com/apache/helix/pull/2222
### Description <!-- Write a concise description: "what?, why?, how?" and then add some details about this PR, including screenshots of any UI changes --> <!-- This PR fixes this Helix issue & includes the Helix issue in the PR description. Link your issue number here: You can write `Fix #123`. See https://docs.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue --> This WIP PR ensures that the response send to helix-rest from helix-front is identical to a working curl request. Here is the desired payload format, modeled after the working curl request: ```json {"id":"my-test-cluster","mapFields":{"TEST_FIELD":{"test_sub_field": 18}}} ``` Here is the bug format seen before, which differed slightly from the curl request: ```json { "id":"my-test-cluster", "mapFields":{"TEST_FIELD":{"test_sub_field": "18"}}, "listFields": {}, "simpleFields": {} } ``` Notice that properties with empty object values are included and that the number value of `"test_sub_field"` is a string, not a number. Fix #2221 ### Tests <!-- List the names of new unit or integration tests --> No new tests yet. Will add a test if this proves to be a frontend issue. ### Code Style <!-- Ensure the PR diff has been formatted using [Prettier](https://prettier.io) --> Formatted using [Prettier](https://prettier.io) <!-- ### Changes that Break Backward Compatibility (Optional) - [ ] My PR contains changes that break backward compatibility or previous assumptions for certain methods or API. They include: --> <!-- Consider including all behavior changes for public methods or API. Also include these changes in merge description so that other developers are aware of these changes. This allows them to make relevant code changes in feature branches accounting for the new method or API behavior. --> <!-- ### Documentation (Optional) - [ ] In case of new functionality, my PR adds documentation in the following wiki page: --> <!-- Link the GitHub wiki you added --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
