Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set maximumSize using @HystrixCommand annotation #1565

Open
ilyas-keser opened this issue May 4, 2017 · 6 comments
Open

Set maximumSize using @HystrixCommand annotation #1565

ilyas-keser opened this issue May 4, 2017 · 6 comments

Comments

@ilyas-keser
Copy link

ilyas-keser commented May 4, 2017

It is not possible to set the maximumSize using the HystrixCommand annotation. Example:

  @HystrixCommand(ignoreExceptions = MyException.class,
    threadPoolProperties = {
      @HystrixProperty(name = "coreSize", value = "20"),
      @HystrixProperty(name = "maximumSize", value = "25"),
      @HystrixProperty(name = "allowMaximumSizeToDivergeFromCoreSize", value = "true")
  })
  public MyPerson getMyPerson(...)

Hystrix version: 1.5.11

Exception:

Caused by: java.lang.IllegalArgumentException: unknown thread pool property: maximumSize
	at com.netflix.hystrix.contrib.javanica.conf.HystrixPropertiesManager.initializeProperties(HystrixPropertiesManager.java:125)
	at com.netflix.hystrix.contrib.javanica.conf.HystrixPropertiesManager.initializeThreadPoolProperties(HystrixPropertiesManager.java:108)
	at com.netflix.hystrix.contrib.javanica.command.GenericSetterBuilder.build(GenericSetterBuilder.java:81)
@mattrjacobs
Copy link
Contributor

@dmgcodevil Any cycles to work on this?

@ilyas-keser Any interest in submitting a PR for this functionality?

@dmgcodevil
Copy link
Contributor

dmgcodevil commented May 9, 2017

PR: #1572

@ilyas-keser
Copy link
Author

@dmgcodevil Is it possible to set allowMaximumSizeToDivergeFromCoreSize? The maximumSize setting only takes effect if you also set the allowMaximumSizeToDivergeFromCoreSize setting.

@mattrjacobs
Copy link
Contributor

@ilyas-keser I just submitted #1584

@ilyas-keser
Copy link
Author

@mattrjacobs Thanks!

@dmgcodevil
Copy link
Contributor

we can close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants