Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-5143

Add a new camel-mybatis statementType of 'InsertList' (similar to SelectOne versus SelectList) to allow for mybatis foreach driven batch insert statements

    XMLWordPrintableJSON

Details

    • Patch Available
    • Unknown

    Description

      The camel-mybatis code will iterate over any list passed in and attempt to insert each item individually, bypassing foreach support in statements like:
      <insert id="batchInsertdataCore" parameterType="java.util.List">
      INSERT INTO CORE_DATA (
      <include refid="dataCoreColumns"/>
      )
      VALUES (
      <foreach item="dataCore" collection="_parameter" open="" close="" separator="),(">
      #

      {dataCore.event_id}

      , #

      {dataCore.start_time_val}

      , #

      {dataCore.end_time_val}

      </foreach>
      )
      </insert>

      This results in mybatis generating the following error even when the route is receiving a list of objects as desired:

          1. Error updating database. Cause: org.apache.ibatis.builder.BuilderException: Error evaluating expression '_parameter'. Return value (dataCore{
            event_id=111222333,
            start_time_val=Thu Mar 01 02:03:04 EST 2001,
            end_time_val=Thu Mar 01 02:03:05 EST 2001,
            }
            ) was not iterable.
          2. The error may exist in mybatis/dataCore.xml
          3. The error may involve dataCore.batchInsertdataCore
          4. The error occurred while executing an update

      ---from camel/trunk/components/camel-mybatis/src/main/java/org/apache/camel/component/mybatis/MyBatisProducer.java —
      private void doInsert(Exchange exchange) throws Exception {
      SqlSessionFactory client = endpoint.getSqlSessionFactory();
      SqlSession session = client.openSession();
      try {
      Object result;
      Object in = exchange.getIn().getBody();
      if (in != null) {
      // lets handle arrays or collections of objects
      Iterator<?> iter = ObjectHelper.createIterator(in);
      while (iter.hasNext()) {
      Object value = iter.next();
      LOG.trace("Inserting: {} using statement: {}", value, statement);
      result = session.insert(statement, value);
      doProcessResult(exchange, result);
      }
      } else {
      LOG.trace("Inserting using statement: {}", statement);
      result = session.insert(statement);
      doProcessResult(exchange, result);
      }
      } finally

      { session.commit(); session.close(); }

      }
      ---from camel/trunk/components/camel-mybatis/src/main/java/org/apache/camel/component/mybatis/MyBatisProducer.java —

      It should be simple to add a new statementType = 'InsertList' (similar to SelectOne versus SelectList) and add it to the switch statement further up in the MyBatisProducer.java code.

      Then a new doInsertBatch would be created, copying the code above and just emoving the iterator related code, simply calling:
      result = session.insert(statement, in);

      Attachments

        1. MyBatisInsertListTest.java
          2 kB
          Aaron Daubman
        2. Patch_for_CAMEL-5143.txt
          7 kB
          Aaron Daubman

        Activity

          People

            davsclaus Claus Ibsen
            daubman Aaron Daubman
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 2h
                2h
                Remaining:
                Remaining Estimate - 2h
                2h
                Logged:
                Time Spent - Not Specified
                Not Specified