<div dir="ltr"><div><div><div>Hi Stefan,<br><br></div>I haven't looked at the PR, the changes only apply to GET requests, correct? In other words, will POST requests to '*/data' (e.g. from feeds) still work?<br><br></div>Thanks,<br></div>Thomas<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-04-02 0:24 GMT+02:00 Stefan Negrea <span dir="ltr"><<a href="mailto:snegrea@redhat.com" target="_blank">snegrea@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Hello Everybody,<br><br></div>I submitted today a PR for a long standing JIRA: <a href="https://issues.jboss.org/browse/HWKMETRICS-24" target="_blank">https://issues.jboss.org/browse/HWKMETRICS-24</a> ; here is a related one: <a href="https://issues.jboss.org/browse/HWKMETRICS-57" target="_blank">https://issues.jboss.org/browse/HWKMETRICS-57</a> , and the PR: <a href="https://github.com/hawkular/hawkular-metrics/pull/473" target="_blank">https://github.com/hawkular/hawkular-metrics/pull/473</a><br></div><div><br></div>The JIRAs recommend splitting '*/data' endpoint in two separate endpoints, '/raw' and '/stats'. There are two issues with the current '*/data'. First, the code was really hard to maintain because it was serving two purposes. Depending on the input parameters, the endpoint would serve raw ingested data in some cases and bucketed results in some other cases. The PR still has the old code (since it got just deprecated in this release) and it's not pretty. The second problem was the actual API interface. There was no simple way to know exactly what you get back from the endpoint because sometimes it would return simple data points and sometimes bucketed data. This was based on the query parameters specified in the request. And to make things worse, some parameters could not be mixed, for example, the user could not limit or order bucketed results; but the documentation had to include all under the same endpoint.<br><br></div>The plan is to deprecate existing '*/data' endpoints in the upcoming release (0.15.0) and remove them in the release after that (0.16.0). That gives enough time (roughly 2 months) for all existing projects to migrate to the newer API. Does anybody see any problem with this timeline?<br><br></div>I expect the transition to be simpler because most of the consumers were using '*/data' with the intent to get bucketed results, which is now '*/stats'. So it is just a simple change in most cases.<br><br></div><div>Are there any projects negatively affected by this change in the long-term? Does the change make sense? Is the API interface for retrieving data easier to understand after the change?<br><br><div><div><div><div><div><br clear="all"><div><div><div><div><div><div><div dir="ltr">Thank you,<br>Stefan Negrea<br><br>Software Engineer<br></div></div></div>
</div></div></div></div></div></div></div></div></div></div></div>
<br>_______________________________________________<br>
hawkular-dev mailing list<br>
<a href="mailto:hawkular-dev@lists.jboss.org">hawkular-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/hawkular-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/hawkular-dev</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>Thomas Segismont<br></div>JBoss ON Engineering Team<br></div></div>
</div>