You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2021/01/08 08:41:34 UTC

[GitHub] [iotdb] kr11 commented on a change in pull request #2444: 调整Method not supported为具体提示

kr11 commented on a change in pull request #2444:
URL: https://github.com/apache/iotdb/pull/2444#discussion_r553810336



##########
File path: jdbc/src/main/java/org/apache/iotdb/jdbc/IoTDBPreparedStatement.java
##########
@@ -181,53 +181,63 @@ public void setArray(int parameterIndex, Array x) throws SQLException {
 
   @Override
   public void setAsciiStream(int parameterIndex, InputStream x) throws SQLException {

Review comment:
       The method `setArray()` is quite similar to other changed methods, why not let `setArray()` throw `PARAMETER_SUPPORTED` too?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org