Alluxio / Community

New Contributor Tasks for Alluxio
20 stars 38 forks source link

Change enum fields in ClientIOOperation.java to be upper case #603

Closed apc999 closed 3 years ago

apc999 commented 3 years ago

To follow Java code convention, it should look like

public enum ClientIOOperation {
  /** The write operation, for writing data for the read operations. */
  WRITE,

  /** Streaming read api, using arrays. */
  READ_ARRAY,
  ...

Note that, these enums are also used in CLI API, to avoid break backward compatibility, The fix should be similar to https://github.com/Alluxio/alluxio/pull/13008/ by adding converter = OperationConverter.class for "--operation"

Originally I created this issue for new contributor, with the assumption this is a local change. It turned out much larger scope required. thus moving the task here to main Alluxio issues. Feel free to assign yourself to this task if you are interested to take it