Adjust privileges: caching needed

Hi Deven, after reading https://github.com/phpmyadmin/phpmyadmin/issues/11597 and looking at PMA_checkRequiredPrivilgesForAdjust() I am convinced that this function needs caching. Can you have a look at PMA_analyseShowGrant() and see how we use these functions: PMA_Util::cacheExists() PMA_Util::cacheGet() PMA_Util::cacheSet() -- Marc Delisle | phpMyAdmin

On Sat, Oct 24, 2015 at 4:47 PM, Marc Delisle <marc@infomarc.info> wrote:
Hi Deven,
after reading https://github.com/phpmyadmin/phpmyadmin/issues/11597 and looking at PMA_checkRequiredPrivilgesForAdjust() I am convinced that this function needs caching.
Yes. Indeed, it seems caching is really needed here.
Can you have a look at PMA_analyseShowGrant() and see how we use these functions:
PMA_Util::cacheExists() PMA_Util::cacheGet() PMA_Util::cacheSet()
Thanks for pointing to the example. I will look into it and make a PR to fix this, soon. -- Regards, Deven Bansod * <f2012316@pilani.bits-pilani.ac.in>*
participants (2)
-
Deven Bansod
-
Marc Delisle