fix(flow): batch builder with type

[discord9]

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

What's changed and what's your intention?

fix a bug in which null type vector builder is used incorrectly

Please explain IN DETAIL what the changes are in this PR and why they are needed:

• previously, some vector builder is used wrong and could be null vector builder when encounter null value, this is fixed now as passing actual type is supported to build vector&batch in flow

Checklist

• I have written the necessary rustdoc comments.
• I have added the necessary unit tests and integration tests.
• This PR requires documentation updates.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[discord9]

Fixing failing tests now

[codecov]

Codecov Report

Attention: Patch coverage is 63.15789% with 63 lines in your changes missing coverage. Please review.

Project coverage is 83.81%. Comparing base (7ea8a44) to head (201b683).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5195      +/-   ##
==========================================
- Coverage   84.07%   83.81%   -0.26%     
==========================================
  Files        1181     1181              
  Lines      219203   219329     +126     
==========================================
- Hits       184291   183832     -459     
- Misses      34912    35497     +585     

Copilot reviewed 5 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (6)

• tests/cases/standalone/common/flow/flow_basic.result: Language not supported
• tests/cases/standalone/common/flow/flow_basic.sql: Language not supported
• src/flow/src/compute/render/map.rs: Evaluated as low risk
• src/flow/src/adapter/node_context.rs: Evaluated as low risk
• src/flow/src/compute/render.rs: Evaluated as low risk
• src/flow/src/compute/render/reduce.rs: Evaluated as low risk

Comments suppressed due to low confidence (2)

src/flow/src/adapter/util.rs:26

• Ensure that the new function 'from_proto_to_data_type' has adequate test coverage.

pub fn from_proto_to_data_type(

src/flow/src/adapter/flownode_impl.rs:34

• [nitpick] The error message provided to with_context is not very descriptive. Consider providing more context about the error.

        .with_context(|_| ExternalSnafu)

[fengjiachun]

LGTM

[fengjiachun]

@discord9 Some conflits