Fix #556 - Implementing and refactoring the KILL statement parser

[iifawzi]

Hey!

This PR extends support of KILL statements. completing #557 and fixing 556

Fixes #556

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.81%. Comparing base (edc7db8) to head (afe9173).
Report is 12 commits behind head on 5.10.x.

Additional details and impacted files

@@             Coverage Diff              @@
##             5.10.x     #565      +/-   ##
============================================
+ Coverage     96.79%   96.81%   +0.02%     
- Complexity     2259     2294      +35     
============================================
  Files            69       70       +1     
  Lines          5174     5248      +74     
============================================
+ Hits           5008     5081      +73     
- Misses          166      167       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kamil-tekiela]

DS_Store are MacOS files so they are not relevant to this project. They should not be ignored at all. You can ignore them locally on your machine.

[iifawzi]

it's easier for future contributors to have it ignored by the repo as we do with .idea, vim, and backup files.

[williamdes]

DS_Store are MacOS files so they are not relevant to this project. They should not be ignored at all. You can ignore them locally on your machine.

I checked, and yes I think the actual consensus is to remove them locally. We do not have it in any repo.

Ref: phpmyadmin/phpmyadmin#15903 (comment)
Can you remove the commit ?

[niconoe-]

For what it worth, LGTM.