rusackas commented on PR #37517: URL: https://github.com/apache/superset/pull/37517#issuecomment-5593737448
@VinayaKumar12 checking back in since this went quiet after my last comment. CI's green now after the rebase, so that part's sorted, but I don't think we ever landed on the actual question: coerce in the control itself, or harden the backend compare, instead of special-casing `server_page_length` by name in the reducer? Would rather fix the class of bug than just this one field. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
