Uploaded image for project: 'Spring Framework'
  1. Spring Framework
  2. SPR-13628

Avoid NPE for anonymous SqlParameter in CallMetaDataContext

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Complete
    • Affects Version/s: 3.2.15, 4.1.8, 4.2.2
    • Fix Version/s: 3.2.16, 4.1.9, 4.2.3
    • Component/s: Data:JDBC
    • Labels:
      None
    • Last commented by a User:
      false

      Description

      The following code uses a StoredProcedureItemReader to connect to an Oracle database and call a stored procedure that returns a ref cursor in a parameter:

      SpringItemReaderTest.java
      package springitemreadertest;
      
      import java.util.Map;
      import javax.sql.DataSource;
      import oracle.jdbc.OracleTypes;
      import org.springframework.batch.item.ExecutionContext;
      import org.springframework.batch.item.database.StoredProcedureItemReader;
      import org.springframework.jdbc.core.ColumnMapRowMapper;
      import org.springframework.jdbc.core.SqlParameter;
      
      public class SpringItemReaderTest {
          public static void main(String[] args) throws Exception {
              DataSource dataSource = new MyDataSource();
              StoredProcedureItemReader<Map<String, Object>> storedProcedureItemReader = new StoredProcedureItemReader<>();
              storedProcedureItemReader.setDataSource(dataSource);
              storedProcedureItemReader.setProcedureName("PROC");
              storedProcedureItemReader.setParameters(new SqlParameter[] { new SqlParameter(/*"p_out",*/ OracleTypes.CURSOR) });
              storedProcedureItemReader.setRefCursorPosition(1);
              storedProcedureItemReader.setRowMapper(new ColumnMapRowMapper());
              storedProcedureItemReader.open(new ExecutionContext());
              storedProcedureItemReader.read();
          }
      }
      

      The stored procedure PROC doesn't do anything useful. Here's its definition:

      procedure PROC (p_out OUT SYS_REFCURSOR)
      is
      begin
        open p_out for select * from dual;
      end;
      

      The class MyDataSource isn't particularly interesting either. It implements javax.sql.DataSource but has default implementations for all methods other than getConnection(). Any DataSource that connects to an Oracle database can be used instead of it.

      When I run this code I get the following stacktrace:

      Exception in thread "main" org.springframework.batch.item.ItemStreamException: Failed to initialize the reader
      	at org.springframework.batch.item.support.AbstractItemCountingItemStreamItemReader.open(AbstractItemCountingItemStreamItemReader.java:147)
      	at springitemreadertest.SpringItemReaderTest.main(SpringItemReaderTest.java:20)
      Caused by: java.lang.NullPointerException
      	at org.springframework.jdbc.core.metadata.CallMetaDataContext.reconcileParameters(CallMetaDataContext.java:319)
      	at org.springframework.jdbc.core.metadata.CallMetaDataContext.processParameters(CallMetaDataContext.java:293)
      	at org.springframework.batch.item.database.StoredProcedureItemReader.openCursor(StoredProcedureItemReader.java:168)
      	at org.springframework.batch.item.database.AbstractCursorItemReader.doOpen(AbstractCursorItemReader.java:406)
      	at org.springframework.batch.item.support.AbstractItemCountingItemStreamItemReader.open(AbstractItemCountingItemStreamItemReader.java:144)
      	... 1 more
      

      Of course, to fix the code, uncomment the parameter name I deliberately commented out. However, if I have unintentionally missed out the parameter name, or specified the parameter name and type the wrong way around (hence setting the type name instead of the parameter name), it would have been nice to have got an exception with a message that told me I needed to set a name for the SqlParameter.

      Incidentally, issue SPR-4814, which I found while seeing if this has already been reported, contains the following comment:

      CallMetaDataContext reconcileParameters should check that a name exists for parameter to avoid NPE

        Attachments

          Activity

            People

            • Assignee:
              juergen.hoeller Juergen Hoeller
              Reporter:
              lukewoodward Luke Woodward
              Last updater:
              Juergen Hoeller
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                3 years, 3 weeks, 1 day ago