Details

      Description

      Test file write/read, syncable, positioned read, seek and others for Crypto streams.

      1. HDFS-6405.patch
        8 kB
        Yi Liu
      2. HDFS-6405.5.patch
        5 kB
        Yi Liu
      3. HDFS-6405.4.patch
        4 kB
        Yi Liu
      4. HDFS-6405.3.patch
        4 kB
        Yi Liu
      5. HDFS-6405.2.patch
        4 kB
        Yi Liu
      6. HDFS-6405.1.patch
        14 kB
        Yi Liu

        Issue Links

          Activity

          Yi Liu created issue -
          Yi Liu made changes -
          Field Original Value New Value
          Parent HDFS-6134 [ 12702810 ]
          Issue Type Test [ 6 ] Sub-task [ 7 ]
          Hide
          Yi Liu added a comment -

          This test case is to verify basic functionality for crypto streams in HADOOP-10603.

          Show
          Yi Liu added a comment - This test case is to verify basic functionality for crypto streams in HADOOP-10603 .
          Yi Liu made changes -
          Attachment HDFS-6405.patch [ 12644929 ]
          Yi Liu made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          Yi Liu added a comment -

          1. Test crypto file writing with different buffer size.
          2. Test crypto file reading with different buffer size.
          3. Test hflush/hsync of crypto output stream, and with different buffer size.
          4. Test positioned read of crypto file.
          5. Test seek to different positions for crypto file.
          6. Test get file position of crypto file.
          7. Test skip operations for crypto file.

          Show
          Yi Liu added a comment - 1. Test crypto file writing with different buffer size. 2. Test crypto file reading with different buffer size. 3. Test hflush/hsync of crypto output stream, and with different buffer size. 4. Test positioned read of crypto file. 5. Test seek to different positions for crypto file. 6. Test get file position of crypto file. 7. Test skip operations for crypto file.
          Yi Liu made changes -
          Attachment HDFS-6405.1.patch [ 12645274 ]
          Yi Liu made changes -
          Affects Version/s fs-encryption (HADOOP-10150 and HDFS-6134) [ 12326852 ]
          Affects Version/s 3.0.0 [ 12320356 ]
          Yi Liu made changes -
          Fix Version/s fs-encryption (HADOOP-10150 and HDFS-6134) [ 12326852 ]
          Fix Version/s 3.0.0 [ 12320356 ]
          Yi Liu made changes -
          Target Version/s 3.0.0 [ 12320356 ] fs-encryption (HADOOP-10150 and HDFS-6134) [ 12326852 ]
          Hide
          Yi Liu added a comment -

          Refine TestHdfsCryptoStreams extends CryptoStreamsTestBase.

          Show
          Yi Liu added a comment - Refine TestHdfsCryptoStreams extends CryptoStreamsTestBase .
          Yi Liu made changes -
          Attachment HDFS-6405.2.patch [ 12645470 ]
          Yi Liu made changes -
          Attachment HDFS-6405.3.patch [ 12645529 ]
          Yi Liu made changes -
          Attachment HDFS-6405.4.patch [ 12646520 ]
          Yi Liu made changes -
          Link This issue relates to HADOOP-10603 [ HADOOP-10603 ]
          Hide
          Yi Liu added a comment -

          HDFS side test cases for Crypto streams.

          Show
          Yi Liu added a comment - HDFS side test cases for Crypto streams.
          Hide
          Andrew Wang added a comment -

          Hi Yi, overall looks good, few small comments:

          • writeCheck has an unused parameter Path file
          • shutdown, put a space after if, feel free to just run the auto-formatter on this file
          • Any reason we couldn't just fold testFileWrite into CryptoStreamsTestBase? We could do an instanceof FSDataOutputStream check. I also see that the subclasses @Ignore out unsupported tests, which is another option.
          Show
          Andrew Wang added a comment - Hi Yi, overall looks good, few small comments: writeCheck has an unused parameter Path file shutdown, put a space after if , feel free to just run the auto-formatter on this file Any reason we couldn't just fold testFileWrite into CryptoStreamsTestBase? We could do an instanceof FSDataOutputStream check. I also see that the subclasses @Ignore out unsupported tests, which is another option.
          Hide
          Yi Liu added a comment -

          Andrew Wang, thanks for your good comments. I update the patch for them.

          Show
          Yi Liu added a comment - Andrew Wang , thanks for your good comments. I update the patch for them.
          Yi Liu made changes -
          Attachment HDFS-6405.5.patch [ 12655997 ]
          Hide
          Andrew Wang added a comment -

          +1 thanks Yi, I committed this to the branch.

          Show
          Andrew Wang added a comment - +1 thanks Yi, I committed this to the branch.
          Andrew Wang made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open In Progress In Progress
          4m 52s 1 Yi Liu 15/May/14 01:16
          In Progress In Progress Resolved Resolved
          62d 18h 47m 1 Andrew Wang 16/Jul/14 20:04

            People

            • Assignee:
              Yi Liu
              Reporter:
              Yi Liu
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development