Skip to content

Commit

Permalink
re-default global max rpc batch size to 1k (#5104) (#5105)
Browse files Browse the repository at this point in the history
* default global max rpc batch size to 1000 for now

Signed-off-by: garyschulte <[email protected]>
  • Loading branch information
garyschulte authored Feb 17, 2023
1 parent 08c3f84 commit 6d6bb21
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 5 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ From the improvements and fixes side, we have a host of execution performance im
One final note. 23.1.0 is not a Shanghai ready release. If you intend to test Besu on the long-lived testnets like Zhejiang, please follow the instructions here. We will have more to share on our official Shanghai releases soon.

### Breaking Changes
- Add a new CLI option to limit the number of requests in a single RPC batch request. Default=1 [#4965](https://github.com/hyperledger/besu/pull/4965)
- Change JsonRpc http service to return the error -32602 (Invalid params) with a 200 http status code
- Besu requires minimum Java 17 and up to build and run [#3320](https://github.com/hyperledger/besu/issues/3320)
- PKCS11 with nss module (PKCS11 based HSM can be used in DevP2P TLS and QBFT PKI) does not work with RSA keys
Expand All @@ -23,6 +22,8 @@ tests are updated to use EC private keys instead of RSA keys.
- Optimize SSTORE Operation execution time (memoize current and original value) [#4836](https://github.com/hyperledger/besu/pull/4836)

### Additions and Improvements
- Default rpc batch request to 1000 [#5104](https://github.com/hyperledger/besu/pull/5104)
- Add a new CLI option to limit the number of requests in a single RPC batch request. [#4965](https://github.com/hyperledger/besu/pull/4965)
- Support for new DATAHASH opcode as part of EIP-4844 [#4823](https://github.com/hyperledger/besu/issues/4823)
- Send only hash announcement for blob transaction type [#4940](https://github.com/hyperledger/besu/pull/4940)
- Add `excess_data_gas` field to block header [#4958](https://github.com/hyperledger/besu/pull/4958)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ public interface DefaultCommandValues {
/** The constant DEFAULT_HTTP_MAX_CONNECTIONS. */
int DEFAULT_HTTP_MAX_CONNECTIONS = 80;
/** The constant DEFAULT_HTTP_MAX_BATCH_SIZE. */
int DEFAULT_HTTP_MAX_BATCH_SIZE = 1;
int DEFAULT_HTTP_MAX_BATCH_SIZE = 1000;
/** The constant DEFAULT_WS_MAX_CONNECTIONS. */
int DEFAULT_WS_MAX_CONNECTIONS = 80;
/** The constant DEFAULT_WS_MAX_FRAME_SIZE. */
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ public class JsonRpcConfiguration {
public static final int DEFAULT_JSON_RPC_PORT = 8545;
public static final int DEFAULT_ENGINE_JSON_RPC_PORT = 8551;
public static final int DEFAULT_MAX_ACTIVE_CONNECTIONS = 80;
public static final int DEFAULT_MAX_BATCH_SIZE = 1;
public static final int DEFAULT_MAX_BATCH_SIZE = 1000;

private boolean enabled;
private int port;
Expand Down Expand Up @@ -208,6 +208,7 @@ public String toString() {
.add("tlsConfiguration", tlsConfiguration)
.add("httpTimeoutSec", httpTimeoutSec)
.add("maxActiveConnections", maxActiveConnections)
.add("maxBatchSize", maxBatchSize)
.toString();
}

Expand All @@ -228,7 +229,8 @@ public boolean equals(final Object o) {
&& Objects.equals(rpcApis, that.rpcApis)
&& Objects.equals(hostsAllowlist, that.hostsAllowlist)
&& Objects.equals(authenticationCredentialsFile, that.authenticationCredentialsFile)
&& Objects.equals(authenticationPublicKeyFile, that.authenticationPublicKeyFile);
&& Objects.equals(authenticationPublicKeyFile, that.authenticationPublicKeyFile)
&& maxBatchSize == that.maxBatchSize;
}

@Override
Expand All @@ -242,7 +244,8 @@ public int hashCode() {
hostsAllowlist,
authenticationEnabled,
authenticationCredentialsFile,
authenticationPublicKeyFile);
authenticationPublicKeyFile,
maxBatchSize);
}

public int getMaxActiveConnections() {
Expand Down

0 comments on commit 6d6bb21

Please sign in to comment.