jax-rs-spec
  1. jax-rs-spec
  2. JAX_RS_SPEC-388

BeanValidation integration in the spec needs update to reflect recent BV API changes.

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: spec
    • Labels:
      None

      Description

      From BV spec lead:

      We discovered a flaw in Bean validation's @ValidateExecutable design
      that could cause proper CDI implementations to be quite inefficient at
      deployment time unless some very advanced class loader scanning is in
      place.
      Since we discussed deployment time efficiency from the get go for method
      validation, we have decided to change the design to accommodate for more
      efficient designs implementable much more easily.

      The end result is that @ValidateExecutable is slightly updated (with a
      new default value IMPLICIT) and we are strongly considering renaming it.

      AFAIK that means a small change in the JAX-RS spec (due to the rename)
      and a change in your RI which I think will be limited but we all know so
      late in the game every change is tough.

        Activity

        Hide
        emmanuelbernard added a comment -

        The changes are up at http://beanvalidation.org/latest-draft/spec/#integration-general-executable

        I anticipate to release the API jar as 1.1.0.CR2 tomorrow morning.

        To know exactly what has changed, look at

        https://github.com/beanvalidation/beanvalidation-spec/commit/3b5380ad3951975e6808915438f8f6662c7a13a5
        and https://github.com/beanvalidation/beanvalidation-spec/commit/cf12e9d0339e7fa1c5d81cd040e5b47f49544edc

        The former is the semantic change, the latter is the renaming.

        Show
        emmanuelbernard added a comment - The changes are up at http://beanvalidation.org/latest-draft/spec/#integration-general-executable I anticipate to release the API jar as 1.1.0.CR2 tomorrow morning. To know exactly what has changed, look at https://github.com/beanvalidation/beanvalidation-spec/commit/3b5380ad3951975e6808915438f8f6662c7a13a5 and https://github.com/beanvalidation/beanvalidation-spec/commit/cf12e9d0339e7fa1c5d81cd040e5b47f49544edc The former is the semantic change, the latter is the renaming.
        Hide
        Santiago Pericas-Geertsen added a comment -

        Updated section on default validation mode and @ValidateOnExecution.

        Show
        Santiago Pericas-Geertsen added a comment - Updated section on default validation mode and @ValidateOnExecution.

          People

          • Assignee:
            Santiago Pericas-Geertsen
            Reporter:
            Marek Potociar
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: