8000 Implement batch webhook concurency by cmiacz · Pull Request #18323 · saleor/saleor · GitHub
[go: up one dir, main page]
Skip to content

Implement batch webhook concurency #18323

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 6 commits into
base: eng-860-wh-mutex
Choose a base branch
from
Draft

Implement batch webhook concurency #18323

wants to merge 6 commits into from

Conversation

@cmiacz
Copy link
Member
@cmiacz cmiacz commented Oct 16, 2025

I want to merge this change because...

Impact

  • New migrations
  • New/Updated API fields or mutations
  • Deprecated API fields or mutations
  • Removed API types, fields, or mutations

Docs

  • Link to documentation:

Pull Request Checklist

  • Privileged queries and mutations are either absent or guarded by proper permission checks
  • Database queries are optimized and the number of queries is constant
  • Database migrations are either absent or optimized for zero downtime
  • The changes are covered by test cases
  • All new fields/inputs/mutations have proper labels added (ADDED_IN_X, PREVIEW_FEATURE, etc.)
  • All migrations have proper dependencies
  • All indexes are added concurrently in migrations
  • All RunSql and RunPython migrations have revert option defined

@cmiacz cmiacz added the load test Deploy to load test infra label Oct 16, 2025
@db-queries
Copy link
db-queries bot commented Oct 16, 2025

Here is the report for 7d70168 (saleor:eng-788-wh-batch-concurency)
Missing base report (cc68e39). The results couldn't be compared.

No differences were found. 







      		
    

  





        


            

            
            

              
            

        

      


      

            
  
  

    
  
    
    

  

  

  


    












      

  
        





      

  
            

    

      
    

    


      


          @cmiacz
cmiacz




          removed
  the 

  load test

  Deploy to load test infra
 label


      Oct 28, 2025

    

  












  
  

    

      

  




      


    


    

      

        
      

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



      

    

  




  
          



      

  

    
    

    

  

    Reviewers
  


    

    No reviews






    

    

  


      
  

    Assignees
  



        


    No one assigned







      


  


  

    Labels
  



    

    None yet







      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
  

    Development
  



            


Successfully merging this pull request may close these issues.





  
  



      
    

  




      

    

  
      

  

    

      2 participants
    

    

        
          @cmiacz 
        
           
    

  






    


    


    








  

    

  

  


    



  





    

  


  


          



    




  

    

    

    





    

    

    

  



0