Re: [DISCUSS] Remove `/api/v1/update/replace` endpoint

2018-12-07 Thread Michael Miklavcic
+1 to that Nick On Fri, Dec 7, 2018 at 11:13 AM Nick Allen wrote: > Heads up - I have not received a response from anyone for about 3 days. I > am going to take that as no one has a problem with this change. I will > likely merge the PR today. > > > > On Tue, Dec 4, 2018 at 4:05 PM Nick Allen

Re: [DISCUSS] Remove `/api/v1/update/replace` endpoint

2018-12-07 Thread Nick Allen
Heads up - I have not received a response from anyone for about 3 days. I am going to take that as no one has a problem with this change. I will likely merge the PR today. On Tue, Dec 4, 2018 at 4:05 PM Nick Allen wrote: > PR #1284 completely removes the `/api/v1/update/replace` endpoint.

[DISCUSS] Remove `/api/v1/update/replace` endpoint

2018-12-04 Thread Nick Allen
PR #1284 completely removes the `/api/v1/update/replace` endpoint. This endpoint is not being used by any other services in Metron currently. As part of the work I am doing to allow Elasticsearch to auto-generate the document ID, I would have had to make code changes to this endpoint to continue