Skip to content

Commit

Permalink
Fix FUSE deprecated property key
Browse files Browse the repository at this point in the history
### What changes are proposed in this pull request?

Alluxio deprecated property key doesn't work well with new property key
with alias.
Remove the deprecated property key as a workaround and will fix the
general problem in a later PR.

### Why are the changes needed?
When setting the value for deprecated property key,
the new property key with the alias name as the deprecated property key
name does not take the value.

### Does this PR introduce any user facing changes?
Supporting alluxio.worker.fuse.mount.point,
alluxio.worker.fuse.mount.alluxio.path, and
alluxio.worker.fuse.mount.options.

pr-link: #15286
change-id: cid-267beead754ea233d2034e24647e59afbe51dd09
  • Loading branch information
LuQQiu authored Apr 11, 2022
1 parent 4b43e50 commit acf8c52
Showing 1 changed file with 0 additions and 37 deletions.
37 changes: 0 additions & 37 deletions core/common/src/main/java/alluxio/conf/PropertyKey.java
Original file line number Diff line number Diff line change
Expand Up @@ -3393,43 +3393,6 @@ public String toString() {
.setConsistencyCheckLevel(ConsistencyCheckLevel.WARN)
.setScope(Scope.WORKER)
.build();
/**
* @deprecated use {@link #FUSE_MOUNT_ALLUXIO_PATH} instead
*/
@Deprecated
public static final PropertyKey WORKER_FUSE_MOUNT_ALLUXIO_PATH =
stringBuilder(Name.WORKER_FUSE_MOUNT_ALLUXIO_PATH)
.setDefaultValue("/")
.setDescription(format("The Alluxio path to mount to the given "
+ "Fuse mount point configured by %s in this worker.",
Name.WORKER_FUSE_MOUNT_POINT))
.setConsistencyCheckLevel(ConsistencyCheckLevel.WARN)
.setScope(Scope.WORKER)
.build();
/**
* @deprecated use {@link #FUSE_MOUNT_OPTIONS} instead
*/
@Deprecated
public static final PropertyKey WORKER_FUSE_MOUNT_OPTIONS =
listBuilder(Name.WORKER_FUSE_MOUNT_OPTIONS)
.setDescription("The platform specific Fuse mount options "
+ "to mount the given Fuse mount point. "
+ "If multiple mount options are provided, separate them with comma.")
.setConsistencyCheckLevel(ConsistencyCheckLevel.WARN)
.setScope(Scope.WORKER)
.build();
/**
* @deprecated use {@link #FUSE_MOUNT_POINT} instead
*/
@Deprecated
public static final PropertyKey WORKER_FUSE_MOUNT_POINT =
stringBuilder(Name.WORKER_FUSE_MOUNT_POINT)
.setDefaultValue("/mnt/alluxio-fuse")
.setDescription("The absolute local filesystem path that this worker will "
+ "mount Alluxio path to.")
.setConsistencyCheckLevel(ConsistencyCheckLevel.WARN)
.setScope(Scope.WORKER)
.build();
public static final PropertyKey WORKER_STARTUP_TIMEOUT =
durationBuilder(Name.WORKER_STARTUP_TIMEOUT)
.setDefaultValue("10min")
Expand Down

0 comments on commit acf8c52

Please sign in to comment.