gortiz opened a new pull request, #12200:
URL: https://github.com/apache/pinot/pull/12200

   We recently find issues using the `resumeConsumption` endpoint and we think 
it would be better to improve the swagger usability. Specifically, in current 
version the swagger UI is:
   
   
![image](https://github.com/apache/pinot/assets/1913993/e97d218f-567c-43f3-add3-91ca59ae6722)
   
   It is important to notice that the expected and safe value for `consumeFrom` 
is null. That is explained in the description, but a misconfiguration may 
produce data loss, we think it is important to improve the message and 
usability. This PR does the following:
   1. Adds a new value for `consumeFrom` called `lastUsed`. When that value is 
provided, Pinot behaves exactly as when null is provided.
   2. The UX is improved in such a way that:
      * The list of valid values is codified, so swagger shows a combo box to 
select them
      * Each value has a small description of its effect
   
   As a result, this is the new UI:
   
![image](https://github.com/apache/pinot/assets/1913993/4f3ac7c6-896f-414f-9958-9c2970d4bf7f)
   


-- 
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]

Reply via email to