[concurrency][distributed] Inject invocations of transport.actorReady in init #39762
                        
                     
                 
             
            
                
                    
                        
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                            
                            
                                
                                
                                    
                                        
                                            
                                            
                                            
                                        
                                        
                                            
        ktoso
    
                                            
                                            
                                            commented
                                            
                                            on 2021-10-19
                                            
                                         
                                     
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                 
                             
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            impose `self` restrictions in distributed actor inits
                                         
                                        
                                            cecdc807
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            Replace `destoryDistributedActor` builtin with `destroyDefaultActor`
                                         
                                        
                                            006e2b44
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            fixed emission of actorReady call in init epilogue
                                         
                                        
                                            38e6303d
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            refactor some of SILGen's distributed actor code
                                         
                                        
                                            8d3c9b0e
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            invoke resignIdentity in an async failable initializer
                                         
                                        
                                            b7d5e0af
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            decouple the emission of actorReady from SILGen
                                         
                                        
                                            51b76979
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            inject actorReady calls for async dist actor ctors
                                         
                                        
                                            e54fa6c6
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            update dist actor deinit SIL tests
                                         
                                        
                                            0651acea
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            merge coverage of multiple transport args into a runtime test
                                         
                                        
                                            cc1ae9f9
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            Move the few SIL tests under the Distributed directory
                                         
                                        
                                            bdd87669
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            add SIL coverage for current actorReady and resignIdentity calls
                                         
                                        
                                            efe8f5de
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            remove redundant defers, now that actorReady is called implicitly
                                         
                                        
                                            e545557d
                                        
                                     
                                 
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            capture the known problem with multiple actorReady calls in async inits
                                         
                                        
                                            f75a6e94
                                        
                                     
                                 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    force pushed
                                    
                                    
                                    to
                                    f75a6e94
                                    
                                    4 years ago
                                
         
     
 
                            
                        
                            
                            
                                
                                
                                    
                                        
                                            
                                            
                                            
                                        
                                        
                                            
        ktoso
    
                                            
                                            
                                            approved these changes
                                            
                                            on 2021-10-22
                                            
                                         
                                     
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                    
                                 
                             
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            throwing sync inits are also not fully working
                                         
                                        
                                            9a76323b
                                        
                                     
                                 
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                marked this pull request as ready for review 4 years ago
                                
         
     
 
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                            
                                
                                    
                                    
                                        
                                        
                                            
                                            
        
    
                                            
                                            fix whitespace in check line for Linux
                                         
                                        
                                            191c996d
                                        
                                     
                                 
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                            
                                
    
    
        
        
            
            
        
    
            
        kavon
    
            
            
                                    merged
                                    
                                    6061de21
                                    
                                    into main 4 years ago
                                
         
     
 
                            
                        
                            
                            
                            
                        
                            
                            
                            
                        
                     
                    
                    
                    
                 
                
                    
                    
                        Assignees
                        
                        
                            No one assigned
                        
                        
                     
                    
                    
                 
             
         
        
        
     
 
     
Login to write a write a comment.
Login via GitHub