HuangXingBo opened a new pull request, #21430:
URL: https://github.com/apache/flink/pull/21430

   ## What is the purpose of the change
   
   *This pull request will change some configs to make grpc server stable in 
PyFlink Process Mode.*
   
   
   ## Brief change log
   
     - *Port Beam `ServerFactory` class to flink-python module*
     - *Change default `KEEP_ALIVE_TIME_SEC` to 19. The BDP ping period is 
locally-decided and the keep alive time is 20 seconds in client side, so we 
choose the server to allow pings every 19 seconds. For details, you can refer 
to the discussion in https://github.com/grpc/grpc/issues/17667*
     - *Adjust the default value of bundle size and arrow batch size. The 
default value of bundle size is 100000. But in most scenarios, this default 
value is unreasonable, and it is easy to cause instability of the grpc server, 
so we adjust the default value to 1000.*
   
   ## Verifying this change
   
   This change added tests and can be verified as follows:
   
     - *Original tests*
     - *Manual test a job which takes 1min in a udf*
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
     - The serializers: (no)
     - The runtime per-record code paths (performance sensitive): (no)
     - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
     - The S3 file system connector: (no)
   
   ## Documentation
   
     - Does this pull request introduce a new feature? (no)
     - If yes, how is the feature documented? (not applicable)
   


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to