jixuan1989 commented on a change in pull request #34: fix sonar
URL: https://github.com/apache/incubator-iotdb/pull/34#discussion_r251851500
##########
File path:
jdbc/src/main/java/org/apache/iotdb/jdbc/ResultSetMetaDataAdapter.java
##########
@@ -0,0 +1,103 @@
+package org.apache.iotdb.jdbc;
+
+import java.sql.ResultSetMetaData;
+import java.sql.SQLException;
+
+public abstract class ResultSetMetaDataAdapter implements ResultSetMetaData {
Review comment:
> 这个类加了干什么？
Using Adaptor to avoid an interface's implementation classes writing too
many unneeded functions is common. But because we only has one implementation
class of `ResultSetMetaData`, I think adding an Adaptor class is not essential.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
