Camel
  1. Camel
  2. CAMEL-4409

Add unit test to camel-sql to ensure deterministic column order

    Details

    • Type: Test Test
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.8.2, 2.9.0
    • Component/s: camel-sql
    • Labels:
      None
    • Estimated Complexity:
      Novice

      Description

      This is already working but not covered by a unit test.
      CAMEL-4350 fixed column ordering for camel-jdbc & got me looking at camel-sql, the patch is almost ready.

        Activity

        Hide
        Mathieu Lalonde added a comment -

        Here is a unit test to ensure deterministic key ordering.

        Since the change was so trivial, I also removed warnings & fixed checkstyle violations within the same patch.

        Show
        Mathieu Lalonde added a comment - Here is a unit test to ensure deterministic key ordering. Since the change was so trivial, I also removed warnings & fixed checkstyle violations within the same patch.
        Hide
        Claus Ibsen added a comment -

        Thanks Mathieu.

        Even checkstyle was okay

        Show
        Claus Ibsen added a comment - Thanks Mathieu. Even checkstyle was okay

          People

          • Assignee:
            Mathieu Lalonde
            Reporter:
            Mathieu Lalonde
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development