Skip to content

Conversation

@omalley
Copy link
Contributor

@omalley omalley commented Jun 17, 2019

The PhysicalWriter and its implementation need a couple of changes:

  • Stripe & file statistics need to be handled differently to support encryption. There is now a method on the WriterContext that allows setting the stripe & file statistics.
  • getFileBytes needs to know which encryption variant is being asked about
  • I generalized PhysicalWriter.getCompressionCodec to getStreamOptions.
  • The PhysicalFsWriter now tracks each encryption variant and handles them separately.
  • StreamName gets the encryption variant.
  • TreeWriter gets a new method for getting the current file stats.
  • Add WriterEncryptionVariant and WriterEncryptionKey to keep encryption information.

omalley added a commit to omalley/orc that referenced this pull request Jun 17, 2019
omalley added a commit to omalley/orc that referenced this pull request Jun 17, 2019
@omalley omalley merged commit df79ad1 into apache:master Jun 25, 2019
zratkai pushed a commit to zratkai/orc that referenced this pull request Jun 20, 2024
Fixes apache#402

Signed-off-by: Owen O'Malley <[email protected]>
Change-Id: I67ebedc34bf08811a369b2da2050c1664b787fb6
zratkai pushed a commit to zratkai/orc that referenced this pull request Jun 20, 2024
Fixes apache#402

Signed-off-by: Owen O'Malley <[email protected]>
Change-Id: I97cfa698a6f3fcadcd202adbb2f6a3f26e696308
zratkai pushed a commit to zratkai/orc that referenced this pull request Jun 20, 2024
Fixes apache#402

Signed-off-by: Owen O'Malley <[email protected]>
Change-Id: I97cfa698a6f3fcadcd202adbb2f6a3f26e696308
zratkai pushed a commit to zratkai/orc that referenced this pull request Jun 20, 2024
Fixes apache#402

Signed-off-by: Owen O'Malley <[email protected]>
Change-Id: I4e1dbd8b81505a66af45ebb1abe49537a4842457
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant