[ARROW-17647] [C++] Using better namespace style when using protobuf with Substrait

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 10.0.0
• Component/s: C++
• Labels: pull-request-available substrait
• External issue URL: https://github.com/apache/arrow/issues/20401
• Assignee: Vibhatha Lakmal Abeykoon
• Reporter: Vibhatha Lakmal Abeykoon
• Created: 08/Sep/22 00:52
• Updated: 11/Jan/23 11:52
• Resolved: 19/Sep/22 22:54

Description

At the moment the namespace usage in Substrait is not consistent when it comes to using Protobuf classes in serialization and deserialization tasks. 

 

For example: 

Instead of `substrait::ReadRel_LocalFiles_FileOrFiles` use `substrait::ReadRel::LocalFiles::FileOrFiles`

 

Issue Links

• links to: GitHub Pull Request #14121