Skip to content

Conversation

@jkff
Copy link
Contributor

@jkff jkff commented Sep 1, 2017

This is on top of #3759.

  • Creates a ReadableFile type that's just a utility wrapper over a Metadata and Compression.
  • Creates FileIO.read() that returns ReadableFile's - this subsumes BEAM-2750.
  • Creates versions readFiles() in TextIO and XmlIO that give access to all the features of FileIO to all users of these IOs. For example, XmlIO does not explicitly support watching for new files, or value providers - but you can get them by combining FileIO.match, FileIO.read, and XmlIO.readFiles.

R: @reuvenlax

@jkff jkff force-pushed the readable-file branch 2 times, most recently from 392e797 to ba7fc0d Compare September 1, 2017 23:49
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 69.839% when pulling ba7fc0d on jkff:readable-file into 3aa2bef on apache:master.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 79cba3f on jkff:readable-file into ** on apache:master**.

@asfgit asfgit merged commit abda38d into apache:master Sep 3, 2017
asfgit pushed a commit that referenced this pull request Sep 3, 2017
@jkff jkff deleted the readable-file branch September 3, 2017 23:49
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling abda38d on jkff:readable-file into ** on apache:master**.

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.

4 participants