Honor memcached key size constraints

 






 

 


  Review Request #975
  —
  Created Aug. 19, 2009 and discarded

 


















 

  

   Information

  





 
 

  thilo.ginkel
 









 
 

  Review Board SVN (deprecated)
 









 
 

  
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  reviewboard
 









 
 

  
 






 








 





  


 
 

  
RB will happily try to enter keys into memcached that exceed memcached's key size constraint (250 bytes). This patch should correct this.

 








  


 
 

  
Runs flawlessly on a local RB installation.

 







 









 





  








 









 






  

 








 














 





 

  

   




chipx86




   

   





  

 



 

  


   

   

    
 
chipx86


    
   


  

  

  



    




  1. 
 
    
  
    
   
    I like the concept of the change, but this would be better in the cache_memoize function in Djblets (djblets/util/misc.py).
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Good point. I will update the code and submit an update.
      
 
      

      2. 


 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3.