GitHub user GJL opened a pull request:

    https://github.com/apache/flink/pull/5685

    [FLINK-7521][flip6]

    ## What is the purpose of the change
    
    *Make HTTP request and response limits configurable. A relatively high 
default value is chosen (100 mb) because Netty does not allocate the upper 
limit at once.*
    
    
    ## Brief change log
    
      - *Make HTTP request and response limits configurable.*
    
    ## Verifying this change
    
    This change added tests and can be verified as follows:
      - *Added tests to `RestServerEndpointITCase`*
      - *Manually verified that client and server limits are respected.*
    
    ## Does this pull request potentially affect one of the following parts:
    
      - Dependencies (does it add or upgrade a dependency): (yes / **no**)
      - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes / **no**)
      - The serializers: (yes / **no** / don't know)
      - The runtime per-record code paths (performance sensitive): (yes / 
**no** / don't know)
      - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / **no** / don't know)
      - The S3 file system connector: (yes / **no** / don't know)
    
    ## Documentation
    
      - Does this pull request introduce a new feature? (yes / **no**)
      - If yes, how is the feature documented? (not applicable / docs / 
JavaDocs / **not documented**)


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/GJL/flink FLINK-7521-2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/5685.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #5685
    
----
commit aef8fa247a4c8b14f4dd7ce8f324ccd89bb2ce14
Author: zjureel <zjureel@...>
Date:   2017-09-07T02:39:39Z

    [FLINK-7521] Add config option to set the content length limit of REST 
server and client

commit ff6c7eb1127ff1870f479c1b779379cc22c9dc87
Author: gyao <gary@...>
Date:   2018-03-12T14:44:27Z

    [FLINK-7521][flip6] Remove RestServerEndpoint#MAX_REQUEST_SIZE_BYTES

commit a14e5935dd9132ddb43e55e357674d390ff9c597
Author: gyao <gary@...>
Date:   2018-03-12T22:16:25Z

    [FLINK-7521][flip6] Return HTTP 413 if request limit is exceeded.
    
    Remove unnecessary PipelineErrorHandler from RestClient.
    Rename config keys for configuring request and response limits.
    Set response headers for all error responses.

----


---

Reply via email to