Skip to content
Snippets Groups Projects

Create partition should not return partition info from payload. Empty response is good enough.

Merged Sviatoslav Nekhaienko requested to merge create-no-payload into master

Changes:

  • PartitionInfor is not returned after the creation of a partition, now empty
  • Creation response status is changed from 200(OK) to 201(CREATED)

Please check all calls to create a partition in all services (if any)

Edited by Sviatoslav Nekhaienko

Merge request reports

Merge request pipeline #10760 passed

Pipeline: Partition

#10761

    Merge request pipeline passed for acf075e2

    Deployed to A‎WS‎ 4 years ago

    Merged by ethiraj krishnamanaiduethiraj krishnamanaidu 4 years ago (Oct 1, 2020 5:29pm UTC)

    Loading

    Pipeline #10787 passed

    Pipeline passed for cf959a19 on master

    Deployed to A‎WS‎ 4 years ago

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
    118 118 responses:
    119 119 '200':
    120 120 description: OK
    121 schema:
    122 $ref: '#/definitions/PartitionInfo'
    123 121 '201':
    124 122 description: Created
  • added 1 commit

    • 8abc123e - change the creation return status from 200(OK) to 201(CREATED)

    Compare with previous version

  • Sviatoslav Nekhaienko changed the description

    changed the description

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Duvelis Carao approved this merge request

    approved this merge request

  • Alok Joshi approved this merge request

    approved this merge request

  • ethiraj krishnamanaidu approved this merge request

    approved this merge request

  • @snekhaienko Can you please rebase?

  • added 11 commits

    • b81e52ed - merge from OSDU-delfi
    • dabaf8a5 - remove redis from README.md
    • bc7b9985 - AWS Implementation of Partition Service
    • e1aa541e - Cleanup code and better error handling
    • 4ff6d6cf - fix unit test
    • e1b8a587 - set port to use APPLICATION_PORT env
    • 46b804e1 - Update app properties
    • 12f7b04d - accept HTTP requests since service runs behind ELB
    • 257f48e1 - Set ENVIRONMENT variable for int tests
    • a75b00f8 - allow 10 retries for check if partition was created fully
    • 409515e7 - enable AWS CI

    Compare with previous version

  • added 15 commits

    Compare with previous version

  • Got latest changes from master & rebased

  • ethiraj krishnamanaidu approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading