Re: Accidental kill in UI

2015-01-09 Thread Nicholas Chammas
As Sean said, this definitely sounds like something worth a JIRA issue (and
PR).

On Fri Jan 09 2015 at 8:17:34 AM Sean Owen so...@cloudera.com wrote:

 (FWIW yes I think this should certainly be a POST. The link can become
 a miniature form to achieve this and then the endpoint just needs to
 accept POST only. You should propose a pull request.)

 On Fri, Jan 9, 2015 at 12:51 PM, Joe Wass jw...@crossref.org wrote:
  So I had a Spark job with various failures, and I decided to kill it and
  start again. I clicked the 'kill' link in the web console, restarted the
 job
  on the command line and headed back to the web console and refreshed to
 see
  how my job was doing... the URL at the time was:
 
  /stages/stage/kill?id=1terminate=true
 
  Which of course terminated the stage again. No loss, but if I'd waited a
 few
  hours before doing that, I would have lost data.
 
  I know to be careful next time, but isn't 'don't modify state as a
 result of
  a GET request' the first rule of HTTP? It could lead to an expensive
  mistake. Making this a POST would be a simple fix.
 
  Does anyone else think this is worth creating an issue for?
 
 

 -
 To unsubscribe, e-mail: user-unsubscr...@spark.apache.org
 For additional commands, e-mail: user-h...@spark.apache.org




Accidental kill in UI

2015-01-09 Thread Joe Wass
So I had a Spark job with various failures, and I decided to kill it and
start again. I clicked the 'kill' link in the web console, restarted the
job on the command line and headed back to the web console and refreshed to
see how my job was doing... the URL at the time was:

/stages/stage/kill?id=1terminate=true

Which of course terminated the stage again. No loss, but if I'd waited a
few hours before doing that, I would have lost data.

I know to be careful next time, but isn't 'don't modify state as a result
of a GET request' the first rule of HTTP? It could lead to an expensive
mistake. Making this a POST would be a simple fix.

Does anyone else think this is worth creating an issue for?


Re: Accidental kill in UI

2015-01-09 Thread Sean Owen
(FWIW yes I think this should certainly be a POST. The link can become
a miniature form to achieve this and then the endpoint just needs to
accept POST only. You should propose a pull request.)

On Fri, Jan 9, 2015 at 12:51 PM, Joe Wass jw...@crossref.org wrote:
 So I had a Spark job with various failures, and I decided to kill it and
 start again. I clicked the 'kill' link in the web console, restarted the job
 on the command line and headed back to the web console and refreshed to see
 how my job was doing... the URL at the time was:

 /stages/stage/kill?id=1terminate=true

 Which of course terminated the stage again. No loss, but if I'd waited a few
 hours before doing that, I would have lost data.

 I know to be careful next time, but isn't 'don't modify state as a result of
 a GET request' the first rule of HTTP? It could lead to an expensive
 mistake. Making this a POST would be a simple fix.

 Does anyone else think this is worth creating an issue for?



-
To unsubscribe, e-mail: user-unsubscr...@spark.apache.org
For additional commands, e-mail: user-h...@spark.apache.org