refactor!: isolate arrow code

[varshith257]

Please be sure to look over the pull request guidelines here: https://github.com/spaceandtimelabs/sxt-proof-of-sql/blob/main/CONTRIBUTING.md#submit-pr.

Please go through the following checklist

• The PR title and commit messages adhere to guidelines here: https://github.com/spaceandtimelabs/sxt-proof-of-sql/blob/main/CONTRIBUTING.md. In particular ! is used if and only if at least one breaking change has been introduced.
• I have run the ci check script with source scripts/run_ci_checks.sh.

Rationale for this change

Currently, arrow is one of the elements that must be disabled to enable no_std support. There are many instances of #[cfg(feature = "arrow")] throughout the codebase, which leaves the code cluttered. To make code readable and manageable , arrow module is introduced.

Closes #237
/claim #237

What changes are included in this PR?

• Moved all [cfg(feature = "arrow")] related code to separate modules and cracked cyclic dependencies along the way without any massive disruption.
• Bumped Arrow from 51 -> 53

Are these changes tested?

Yes

[varshith257]

@JayWhite2357 Have a review in your free time.

[iajoiner]

@varshith257 Really thanks for your PR! Actually it would be better if you bump Arrow to 53.1.0 in a separate PR. Or I can do it.

[varshith257]

@iajoiner I will bump to its latest version in follow-up PR if needee

[JayWhite2357]

Looks pretty good so far. The main concern I have is around cyclic dependencies.

The resulting code must not introduce cyclic dependencies and should resolve any existing ones with arrow related code. This is one of the primary reasons for this cleanup: in order to more easily identify and resolve these.

In particular, nothing in base should depend on anything outside of base.

I've added comments to this effect. If you resolve those along with @iajoiner's concerns, we should be good on this PR.

[varshith257]

@JayWhite2357 Will bump with the fixes

[varshith257]

@JayWhite2357 @iajoiner Can have a look at it here?
https://github.com/spaceandtimelabs/sxt-proof-of-sql/actions/runs/11417714964/job/31770330872#step:4:766

I am not sure what it making to fail

[melsonic]

@JayWhite2357 @iajoiner Can have a look at it here? https://github.com/spaceandtimelabs/sxt-proof-of-sql/actions/runs/11417714964/job/31770330872#step:4:766

I am not sure what it making to fail

I believe It's because of being unable to find the implementations in crates/proof-of-sql/src/base/arrow/arrow_query_result.rs.

[JayWhite2357]

I am not sure what it making to fail

@varshith257

replace

RecordBatch::try_from(query_result).unwrap()

with

RecordBatch::try_from(query_result.table).unwrap()

This is caused because I asked you to remove this code:

impl<S: Scalar> TryFrom<QueryData<S>> for RecordBatch {
    type Error = ArrowError;

    fn try_from(value: QueryData<S>) -> Result<Self, Self::Error> {
        Self::try_from(value.table)
    }
}

[iajoiner]

@varshith257 Really thanks for your great work!