Update Base64 as a non-throwing API

[Joe-Abraham]

Follow-up PR: https://github.com/facebookincubator/velox/pull/10371

The following changes are done in this PR.

1. Refactor the APIs in Base64 as non-throwing APIs.

Changing the char to std::string_view and std::string will be done in subsequent PR.

[netlify[bot]]

✅ Deploy Preview for meta-velox canceled.

Name	Link
🔨 Latest commit	03fc98d2760c16006398cb6a1e51f8e19a566599
🔍 Latest deploy log	https://app.netlify.com/sites/meta-velox/deploys/67e6727d1aaa8c00083709b7

[Joe-Abraham]

@majetideepak Can you please review the changes?

[kgpai]

Thanks @Joe-Abraham , based on brief glance, it seems its mostly renaming variables etc for readability and consistency (which is welcome change); Want to make sure I am not missing any material change.

[Joe-Abraham]

@kgpai I have also made the APIs used in BinaryFunctions.h as non-throwing.

[majetideepak]

3. Refactor the APIs in Base64 as non-throwing APIs.

@Joe-Abraham this should be its own PR since its a functional change. We can leave the remaining code improvements this PR. Can you please open another PR with only the non-throwing API changes? Thanks.

[majetideepak]

@Joe-Abraham can you add a brief description of how this change can help with performance? I think Masha's original motivation was to see benefits similar to TRY(CAST()) as explained here https://velox-lib.io/blog/optimize-try-more/

[facebook-github-bot]

@bikramSingh91 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@bikramSingh91 merged this pull request in facebookincubator/velox@c9bbd66230f2714015b63b220a5983b59cf5a2d3.