[ALLUXIO-3222] Change LOAD TtlAction default value to FREE #7425

[...truncated 4151 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/pom.xml to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.8.0-SNAPSHOT/alluxio-core-client-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.8.0-SNAPSHOT/alluxio-core-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.8.0-SNAPSHOT/alluxio-tests-1.8.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

AmplabJenkins · 2018-06-20T08:19:57Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-20T08:19:58Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20277/

Build result: FAILURE

[...truncated 4142 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/pom.xml to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.8.0-SNAPSHOT/alluxio-core-client-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.8.0-SNAPSHOT/alluxio-core-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.8.0-SNAPSHOT/alluxio-tests-1.8.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

Reidddddd · 2018-06-20T08:26:43Z

ping @calvinjia @apc999
Any hint what this failed info indicates?

aaudiber · 2018-06-20T17:52:52Z

@Reidddddd to see the detailed build messages, you can follow the link (https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20277/), then click into "Console Output". The root cause of the failure is usually near the bottom. In this case, it says
"/home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/src/test/java/alluxio/client/fs/TtlIntegrationTest.java:30: error: 'org.junit.Before' should be separated from previous import group."

The coding style section in the website docs gives some advice for how to configure your IDE to automatically put imports in the right order: https://www.alluxio.org/docs/1.7/en/Contributing-to-Alluxio.html#coding-style

AmplabJenkins · 2018-06-21T12:01:21Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-21T12:01:22Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20310/

Failed Tests: 2

org.alluxio:alluxio-tests: 2

Test FAILed.

Reidddddd · 2018-06-21T16:07:09Z

@aaudiber Is there a way to run specific unit test on local? Tried mvn test -Dtest=xxxx, but it didn't work.

aaudiber · 2018-06-21T16:36:25Z

Can you try running mvn test -Dtest=SetTtlCommandIntegrationTest from the tests/ directory?IntelliJ and Eclipse can also let you run individual tests from the UI.

AmplabJenkins · 2018-06-21T16:55:15Z

Merged build finished. Test PASSed.

AmplabJenkins · 2018-06-21T16:55:16Z

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20316/
Test PASSed.

alluxio-bot · 2018-06-21T16:55:19Z

Automated checks report:

Valid pull request title: PASS
Contains link to JIRA ticket: PASS
Commits associated with Github account: PASS

All checks passed!

Reidddddd · 2018-06-22T16:59:47Z

Any comments

aaudiber · 2018-06-22T17:58:17Z

core/common/src/main/java/alluxio/PropertyKey.java

@@ -2416,9 +2416,9 @@ public String toString() {
          .build();
  public static final PropertyKey USER_FILE_CREATE_TTL_ACTION =
      new Builder(Name.USER_FILE_CREATE_TTL_ACTION)
-          .setDefaultValue("DELETE")
+          .setDefaultValue("FREE")


Do you think it makes sense for this to be DELETE by default? For create, the user is creating the file in both Alluxio and the UFS (undo this with DELETE), while for load, the user is only creating the file in Alluxio (undo this with FREE). @calvinjia

IMO free is a safer default since it is non-destructive. The complement for load would be delete, alluxioOnly, which is maybe another TTL mode we should consider.

I see why we would want to use FREE as the default for loading TTLs. DELETE + load can delete any data discovered by Alluxio, even if it was written out of band and isn't aware of Alluxio.

My concern is that users will expect that files created with default TTL will be deleted in both Alluxio and the UFS. That seems like a more natural definition of TTL, and that's how it's worked in Alluxio for the last two years. Have you seen instances where users create files with TTL, then are dismayed when the data is deleted from UFS?

Changed DELETE as default for creation as comment.

And set OutStreamOptions TTL action "DELETE" by default as well, because i think it is related to writing. WDYT.

We should try to maintain backwards compatibility as much as possible so that upgrading to 1.8 doesn't break things for users who have been using TTL in 1.7 and earlier. Previously we didn't have TTLs on load, so we can choose either FREE or DELETE without breaking anything. But for the options that previously included TTL already, we need to stick with DELETE.

I think there's a separate issue where any filesystem operation, including createFile, could trigger files to be loaded from the UFS. In this case, the loaded files should use USER_FILE_LOAD_TTL, while the created file should use USER_FILE_CREATE_TTL. But there's only one TTL in CommonOptions. We can handle that in a separate PR from this one.

So, i should revert all FREE to DELETE except the TTLs for load?

…lAction.DELETE by default

AmplabJenkins · 2018-06-25T17:20:28Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-25T17:20:29Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20347/

Failed Tests: 2

org.alluxio:alluxio-core-client-fs: 2

Test FAILed.

AmplabJenkins · 2018-06-26T04:50:22Z

Merged build finished. Test PASSed.

AmplabJenkins · 2018-06-26T04:50:23Z

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20360/
Test PASSed.

Reidddddd · 2018-06-27T03:23:14Z

Addressed @aaudiber's comment and changed pr's title.

AmplabJenkins · 2018-06-27T03:35:00Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-27T03:35:00Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20374/

Build result: FAILURE

[...truncated 4141 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/pom.xml to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/checker/target/alluxio-checker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-checker/1.8.0-SNAPSHOT/alluxio-checker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.8.0-SNAPSHOT/alluxio-core-client-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.8.0-SNAPSHOT/alluxio-core-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.8.0-SNAPSHOT/alluxio-tests-1.8.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

AmplabJenkins · 2018-06-27T03:50:54Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-27T03:50:55Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20375/

Failed Tests: 1

org.alluxio:alluxio-core-client-fs: 1

alluxio.client.file.BaseFileSystemTest.createFile

Test FAILed.

Reidddddd · 2018-06-27T04:12:26Z

Never modify BaseFileSystemTest.

Reidddddd · 2018-06-27T04:13:08Z

Jenkins, test this please

AmplabJenkins · 2018-06-27T04:35:44Z

Merged build finished. Test FAILed.

AmplabJenkins · 2018-06-27T04:35:45Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20376/

Failed Tests: 1

org.alluxio:alluxio-core-client-fs: 1

alluxio.client.file.BaseFileSystemTest.createFile

Test FAILed.

Reidddddd · 2018-06-27T07:46:55Z

Have no clue in mind about the failed test, please point it out if i missed anything...
And BTW

Can you try running mvn test -Dtest=SetTtlCommandIntegrationTest from the tests/ directory?IntelliJ and Eclipse can also let you run individual tests from the UI.
Not works on my mac...

AmplabJenkins · 2018-06-27T21:05:36Z

Merged build finished. Test PASSed.

AmplabJenkins · 2018-06-27T21:05:37Z

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20387/
Test PASSed.

Reidddddd · 2018-06-28T02:28:56Z

Thanks Andrew!

aaudiber

LGTM

ALLUXIO-3222 Change TtlAction default value to FREE

172aad9

Fix some of the failed UTs

d9cbe00

Fix failed UTs

3a55607

Fix FileSystemMasterTest#setSmallerTtlForDirectoryWithTtl

522f556

Fix checkstyle import group error

aae4a33

Set TtlAction default value to FREE in SetTtlCommand.class

c49969a

alluxio-bot assigned apc999 Jun 21, 2018

aaudiber suggested changes Jun 22, 2018

View reviewed changes

Make client side CreateDirectory(File)Options and OutStreamOptions Tt…

f67812d

…lAction.DELETE by default

Fix LineageFileSystemTest failure

d15225a

Revert all default FREE back to default DELETE

0ef359f

Reidddddd changed the title ~~[ALLUXIO-3222] Change TtlAction default value to FREE~~ [ALLUXIO-3222] Change LOAD TtlAction default value to FREE Jun 27, 2018

Remove unused import

151564c

Fix test

839c9e7

apc999 approved these changes Jun 28, 2018

View reviewed changes

aaudiber approved these changes Jun 28, 2018

View reviewed changes

aaudiber merged commit 69f705f into Alluxio:master Jun 28, 2018

[ALLUXIO-3222] Change LOAD TtlAction default value to FREE #7425

[ALLUXIO-3222] Change LOAD TtlAction default value to FREE #7425

Conversation

Reidddddd commented Jun 19, 2018

Reidddddd commented Jun 19, 2018

AmplabJenkins commented Jun 19, 2018

AmplabJenkins commented Jun 19, 2018

Failed Tests: 18

org.alluxio:alluxio-core-client-fs: 4

org.alluxio:alluxio-core-server-master: 9

org.alluxio:alluxio-tests: 5

AmplabJenkins commented Jun 19, 2018

AmplabJenkins commented Jun 19, 2018

Failed Tests: 14

org.alluxio:alluxio-core-client-fs: 2

org.alluxio:alluxio-core-server-master: 8

org.alluxio:alluxio-tests: 4

AmplabJenkins commented Jun 20, 2018

AmplabJenkins commented Jun 20, 2018

Build result: FAILURE

AmplabJenkins commented Jun 20, 2018

AmplabJenkins commented Jun 20, 2018

Build result: FAILURE

Reidddddd commented Jun 20, 2018 • edited

aaudiber commented Jun 20, 2018

AmplabJenkins commented Jun 21, 2018

AmplabJenkins commented Jun 21, 2018

Failed Tests: 2

org.alluxio:alluxio-tests: 2

Reidddddd commented Jun 21, 2018

aaudiber commented Jun 21, 2018

AmplabJenkins commented Jun 21, 2018

AmplabJenkins commented Jun 21, 2018

alluxio-bot commented Jun 21, 2018

Reidddddd commented Jun 22, 2018

aaudiber Jun 22, 2018

Choose a reason for hiding this comment

calvinjia Jun 22, 2018

Choose a reason for hiding this comment

aaudiber Jun 22, 2018

Choose a reason for hiding this comment

Reidddddd Jun 26, 2018 • edited

Choose a reason for hiding this comment

aaudiber Jun 26, 2018

Choose a reason for hiding this comment

Reidddddd Jun 26, 2018

Choose a reason for hiding this comment

AmplabJenkins commented Jun 25, 2018

AmplabJenkins commented Jun 25, 2018

Failed Tests: 2

org.alluxio:alluxio-core-client-fs: 2

AmplabJenkins commented Jun 26, 2018

AmplabJenkins commented Jun 26, 2018

Reidddddd commented Jun 27, 2018 • edited

AmplabJenkins commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

Build result: FAILURE

AmplabJenkins commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

Failed Tests: 1

org.alluxio:alluxio-core-client-fs: 1

Reidddddd commented Jun 27, 2018

Reidddddd commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

Failed Tests: 1

org.alluxio:alluxio-core-client-fs: 1

Reidddddd commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

AmplabJenkins commented Jun 27, 2018

Reidddddd commented Jun 28, 2018

aaudiber left a comment

Choose a reason for hiding this comment

Reidddddd commented Jun 20, 2018 •

edited

Reidddddd Jun 26, 2018 •

edited

Reidddddd commented Jun 27, 2018 •

edited