This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/main by this push:
new fc96026256 Added NoOpLogCloser for filesystems that don't support
lease recovery (#2860)
fc96026256 is described below
commit fc960262568b2992043520ba9d1ba6cf80305f47
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Wed Aug 10 10:22:34 2022 -0400
Added NoOpLogCloser for filesystems that don't support lease recovery
(#2860)
---
.../server/manager/recovery/NoOpLogCloser.java | 47 ++++++++++++++++++++++
1 file changed, 47 insertions(+)
diff --git
a/server/base/src/main/java/org/apache/accumulo/server/manager/recovery/NoOpLogCloser.java
b/server/base/src/main/java/org/apache/accumulo/server/manager/recovery/NoOpLogCloser.java
new file mode 100644
index 0000000000..e60c847c1d
--- /dev/null
+++
b/server/base/src/main/java/org/apache/accumulo/server/manager/recovery/NoOpLogCloser.java
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.accumulo.server.manager.recovery;
+
+import java.io.IOException;
+
+import org.apache.accumulo.core.conf.AccumuloConfiguration;
+import org.apache.accumulo.core.conf.Property;
+import org.apache.accumulo.server.fs.VolumeManager;
+import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.fs.Path;
+
+/**
+ * {@link HadoopLogCloser} recovers leases in DistributedFileSystem
implementations and does nothing
+ * in local FileSystem implementations. For other implementations {@link
HadoopLogCloser} throws an
+ * exception. However, Accumulo could be used with other Hadoop compatible
FileSystem
+ * implementations that do not support lease recovery and we should not throw
an exception in this
+ * case. This LogCloser implementation supports that case.
+ *
+ * To use this class, set the Property {@link
Property#MANAGER_WAL_CLOSER_IMPLEMENTATION} to the
+ * full name of this class.
+ */
+public class NoOpLogCloser implements LogCloser {
+
+ @Override
+ public long close(AccumuloConfiguration conf, Configuration hadoopConf,
VolumeManager fs,
+ Path path) throws IOException {
+ return 0;
+ }
+
+}
