[cfile] unsorted updates on cfile_reader.{h,cc}

This patch makes the code in cfile_reader.{h,cc} a bit more robust:
  * check more rigorously for invariants to protect against unsigned
    integer underflow, etc.
  * don't crash but instead return Status::Corruption() when possible
  * optimized the check on whether to verify checksums
  * added 'final' to CFileIterator and DefaultColumnValueIterator
    classes to provide more de-virtualisation opportunities for compiler
  * added PREDICT_{FALSE,TRUE} where appropriate
  * added the 'runtime' tag for the --cfile_verify_checksums flag
  * other minor updates

Change-Id: I25507963f87e08a6c3a8ba2ff1ca58836b713e18
Reviewed-on: http://gerrit.cloudera.org:8080/18965
Tested-by: Alexey Serbin <alexey@apache.org>
Reviewed-by: Abhishek Chennaka <achennaka@cloudera.com>
Reviewed-by: Yifan Zhang <chinazhangyifan@163.com>
2 files changed