All override methods must be added @OverRide

Author: slievrly

src/main/java/org/apache/ibatis/datasource/pooled/PooledDataSource.java

@@ -585,19 +585,23 @@ public static Connection unwrapConnection(Connection conn) {
     return conn;
   }
 
+  @Override
   protected void finalize() throws Throwable {
     forceCloseAll();
     super.finalize();
   }
 
+  @Override
   public <T> T unwrap(Class<T> iface) throws SQLException {
     throw new SQLException(getClass().getName() + " is not a wrapper.");
   }
 
+  @Override
   public boolean isWrapperFor(Class<?> iface) {
     return false;
   }
 
+  @Override
   public Logger getParentLogger() {
     return Logger.getLogger(Logger.GLOBAL_LOGGER_NAME);
   }

src/main/java/org/apache/ibatis/datasource/unpooled/UnpooledDataSource.java

@@ -293,6 +293,7 @@ public boolean jdbcCompliant() {
     }
 
     // @Override only valid jdk7+
+    @Override
     public Logger getParentLogger() {
       return Logger.getLogger(Logger.GLOBAL_LOGGER_NAME);
     }
@@ -309,6 +310,7 @@ public boolean isWrapperFor(Class<?> iface) throws SQLException {
   }
 
   // @Override only valid jdk7+
+  @Override
   public Logger getParentLogger() {
     // requires JDK version 1.6
     return Logger.getLogger(Logger.GLOBAL_LOGGER_NAME);

src/main/java/org/apache/ibatis/mapping/SqlCommandType.java

@@ -19,5 +19,5 @@
  * @author Clinton Begin
  */
 public enum SqlCommandType {
-  UNKNOWN, INSERT, UPDATE, DELETE, SELECT, FLUSH;
+  UNKNOWN, INSERT, UPDATE, DELETE, SELECT, FLUSH
 }

src/main/java/org/apache/ibatis/session/Configuration.java

@@ -923,6 +923,7 @@ public StrictMap<V> conflictMessageProducer(BiFunction<V, V, String> conflictMes
       return this;
     }
 
+    @Override
     @SuppressWarnings("unchecked")
     public V put(String key, V value) {
       if (containsKey(key)) {
@@ -940,6 +941,7 @@ public V put(String key, V value) {
       return super.put(key, value);
     }
 
+    @Override
     public V get(Object key) {
       V value = super.get(key);
       if (value == null) {

src/test/java/org/apache/ibatis/autoconstructor/ExtensiveSubject.java

@@ -62,6 +62,6 @@ public ExtensiveSubject(final byte aByte,
     }
 
     public enum TestEnum {
-        AVALUE, BVALUE, CVALUE;
+        AVALUE, BVALUE, CVALUE
     }
-}
+}

src/test/java/org/apache/ibatis/session/SqlSessionTest.java

@@ -583,7 +583,7 @@ void shouldFailSelectOneAuthorUsingMapperClassWithTwoResultHandlers() {
     Configuration configuration = new Configuration(sqlMapper.getConfiguration().getEnvironment());
     configuration.addMapper(AuthorMapperWithMultipleHandlers.class);
     SqlSessionFactory sqlMapperWithMultipleHandlers = new DefaultSqlSessionFactory(configuration);
-    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession();) {
+    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession()) {
       DefaultResultHandler handler1 = new DefaultResultHandler();
       DefaultResultHandler handler2 = new DefaultResultHandler();
       AuthorMapperWithMultipleHandlers mapper = sqlSession.getMapper(AuthorMapperWithMultipleHandlers.class);
@@ -596,7 +596,7 @@ void shouldFailSelectOneAuthorUsingMapperClassWithTwoRowBounds() {
     Configuration configuration = new Configuration(sqlMapper.getConfiguration().getEnvironment());
     configuration.addMapper(AuthorMapperWithRowBounds.class);
     SqlSessionFactory sqlMapperWithMultipleHandlers = new DefaultSqlSessionFactory(configuration);
-    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession();) {
+    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession()) {
       RowBounds bounds1 = new RowBounds(0, 1);
       RowBounds bounds2 = new RowBounds(0, 1);
       AuthorMapperWithRowBounds mapper = sqlSession.getMapper(AuthorMapperWithRowBounds.class);