Spring Projects Issues
Spring Projects Issues
**[Dave Syer](https://jira.spring.io/secure/ViewProfile.jspa?name=david_syer)** commented Agree that it should never be null. We can throw an exception in the setter, but it does seem trivial, and there are plenty of other places...
**[Quentin Bisson](https://jira.spring.io/secure/ViewProfile.jspa?name=qbisson)** commented This issue is also present in the N1qlUtils class: ```java /** * Produces an {@link Expression} that can serve as a WHERE clause criteria to only select...
**[Sani](https://jira.spring.io/secure/ViewProfile.jspa?name=schabiyo)** commented It would be nice to add support for elasticsearch perculator. Thanks in advance
**[Michael Reiche](https://jira.spring.io/secure/ViewProfile.jspa?name=mikereiche)** commented https://github.com/spring-projects/spring-data-couchbase/pull/221
**[Dave Syer](https://jira.spring.io/secure/ViewProfile.jspa?name=david_syer)** commented Agree. And the problem is fairly ubiquitous in afterPropertiesSet() methods. Maybe the change can be scripted?
**[Robert Kasanicky](https://jira.spring.io/secure/ViewProfile.jspa?name=robert.kasanicky)** commented Maybe we should switch to using `@Required` on setters?
**[Dave Syer](https://jira.spring.io/secure/ViewProfile.jspa?name=david_syer)** commented I don't like `@Required`, and I don't think it's appropriate for framework classes, since it requires the user to add a bean post processor for it to...
**[Mahmoud Ben Hassine](https://jira.spring.io/secure/ViewProfile.jspa?name=mbenhassine)** commented @[Iwein Fuld](https://jira.spring.io/secure/ViewProfile.jspa?name=iwein) I also agree. However, there are some places in the code base that use `Assert.state` (like [AbstractJob](https://github.com/spring-projects/spring-batch/blob/master/spring-batch-core/src/main/java/org/springframework/batch/core/job/AbstractJob.java#L116) and [AbstractStep](https://github.com/spring-projects/spring-batch/blob/master/spring-batch-core/src/main/java/org/springframework/batch/core/step/AbstractStep.java#L76-L77)) while other places use `Assert.notNull`...
**[Michael Minella](https://jira.spring.io/secure/ViewProfile.jspa?name=mminella)** commented We have had some conversations about it. No concrete plans yet. We'll keep you posted.
**[Young-Gu Chae](https://jira.spring.io/secure/ViewProfile.jspa?name=chaeyk)** commented I committed code to https://github.com/chaeyk/spring-data-couchbase/commit/6ace51486888505a1a15f24d95d21208b83968fa