ic icon indicating copy to clipboard operation
ic copied to clipboard

chore: replace instruction limits without DTS by instruction limits per query

Open mraszyk opened this issue 3 weeks ago • 0 comments

This PR replaces the limit max_instructions_per_message_without_dts by max_instructions_per_query_message since query methods are now also executed using DTS in the replicated mode of execution.

Furthermore, this PR performs the following changes:

  • max_instructions_per_message_without_dts is ignored by the (replicated call) scheduler since replicated calls now always use DTS;
  • the (replicated call) scheduler does not ignore the limit max_instructions_per_install_code_slice which is high on system subnets and thus affects the computation of round instructions (before this PR max_instructions_per_message_without_dts would dominate so ignoring max_instructions_per_install_code_slice made no difference);
  • default instruction limits for a subnet message are set to zero instead of max_instructions_per_message_without_dts since they are unused (ignored) anyway;
  • fixes instruction limits in ExecutionTest to be production-like by default.

mraszyk avatar Dec 16 '25 10:12 mraszyk