Fix SQL in procedure drilldown report - #2224
Conversation
|
Thank you for finding this issue. I think we could accept this PR off of your master but I would like to caution you about working this way. Instead you should think about branching off master (even in your own fork) to fix changes. So, the workflow is:
This may sound nit-picking, but it will help you out in the future if we are taking multiple PRs into the upstream @anton-abushkevich , I understand you guys are finalizing the release for 2.13. This is a bug that was introduced when fixing the casting problem based on numeric. I just malformed the query in the case of the procedure drilldown. Can this be merged in for 2.13 or did you want to get 2.13 done and fix this in a hotfix? |
yeah, I think you are right,thank you |
Here are two grammatical errors, the source of which was updated on February 16, 2023