pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Validate table name before deletion via User Interface

Open Ehsan-Irshad opened this issue 2 years ago • 4 comments

As a admin when I delete a table on Apache Pinot UI instead of just a confirmation box, I should be requested to validate the table name & type it manually in a dialog box. This is to make sure no wrong table is deleted as sometimes admin might be working with multiple browser tabs

Ehsan-Irshad avatar Mar 17 '23 06:03 Ehsan-Irshad

cc @joshigaurava @jayeshchoudhary

Jackie-Jiang avatar Mar 21 '23 00:03 Jackie-Jiang

Curious if other systems also require something similar?

mayankshriv avatar Mar 21 '23 20:03 mayankshriv

Curious if other systems also require something similar?

I've been thinking about this question for a while. I think for a normal SQL database the answer is, "kind of". You drop a table by typing out DROP TABLE <table_name> where you're effectively doing what's asked here. Github makes you type out the repo name if you're going to delete it. I can't find the post mortem, but I believe AWS and other companies have moved to requiring a second human sign off on (like actively run some command or approve on website) any destructive commands.

All of that said, I think I agree with the ask here. I think anyone running Pinot in prod and relying on the current UI would welcome this feature. If we're worried someone's workflow would really be harmed by this, maybe we can add a "do not ask again for X days" checkbox?

jadami10 avatar Mar 24 '23 15:03 jadami10

After a few more incidents this year where we accidentally deleted the wrong table, I think we also want users to type out the cluster name. If you have a set up where users test tables on a different cluster, it's very easy to accidentally delete the real one instead of the test one.

jadami10 avatar Jun 21 '24 13:06 jadami10