Spring Batch
  1. Spring Batch
  2. BATCH-1535

Extend SQL scripts to provide exit code limit of 100 chars

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.0
    • Fix Version/s: 2.1.1
    • Component/s: Core
    • Labels:
      None

      Description

      Could you please raise the maximum lenght of the exit code to someting a bit bigger than 20 chars.

      I've seen exit status like:

      • COMPLETED WITH ERRORS
      • COMPLETED WITH WARNINGS
      • COMPLETED WITH SKIPS

      For the first one, we put ERROR instead of ERRORS. For the second, we use WARN

      I agree this shouldn't be too big but something like 30 chars allows for better readable exit code. WDYT?

        Activity

        Hide
        Dave Syer added a comment -

        As far as I know there is no restriction in the framework. You can simply update your database schema, right?

        Show
        Dave Syer added a comment - As far as I know there is no restriction in the framework. You can simply update your database schema, right?
        Hide
        Dave Syer added a comment -

        No response from submitter so assume resolvable...

        Show
        Dave Syer added a comment - No response from submitter so assume resolvable...
        Hide
        Stéphane Nicoll added a comment -

        Hey, I haven't seen the notification. Got in my spam for some reason.

        Of course I can but then I need to handle a separate SQL file for it. The purpose of my request was simply to let you know that it could be useful to increase the default value.

        And I can't reopen an issue btw. Weird.

        Show
        Stéphane Nicoll added a comment - Hey, I haven't seen the notification. Got in my spam for some reason. Of course I can but then I need to handle a separate SQL file for it. The purpose of my request was simply to let you know that it could be useful to increase the default value. And I can't reopen an issue btw. Weird.
        Hide
        Dave Syer added a comment -

        Fixed.

        Show
        Dave Syer added a comment - Fixed.

          People

          • Assignee:
            Dave Syer
            Reporter:
            Stéphane Nicoll
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: