Updating to the latest version of DBPS tag

[argmarco-tkd]

Updating to the latest version of DBPS tag

Testing

• All unit tests pass (via ctest -L parquet)
• Successful Manual testing (using correctness verification script) against local + remote instances of DBPA.

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - Contributing Overview

[avalerio-tkd]

Thanks @argmarco-tkd for this and the testing.

Just for my reference, we don't need to update manually any of the common files from DBPS here, right? I had in the back of my head the dbps_interface.h, enums.h, and maybe another, but that's no longer needed, I think?

[avalerio-tkd]

Would we like a named tag for this instead? That may be useful to mark more explicitly the deliverable?

[argmarco-tkd]

We should, but let's have a quick offline chat about this first.

[argmarco-tkd]

Thanks @argmarco-tkd for this and the testing.

Just for my reference, we don't need to update manually any of the common files from DBPS here, right? I had in the back of my head the dbps_interface.h, enums.h, and maybe another, but that's no longer needed, I think?

In short, no, we don't need to do that.
Earlier in the project (before we had the automated setup to download DBPS header files) we had to update DBPS-sourced headers. But now that we have automated downloads we do not need to update those files by hand. The only case where we'd need to update anything other than the tag in CMakeLists.txt is there the interface (contained in dbpa_interface.h was updated - in that case we'd have to make the Arrow code comply with the new version of the interface.

[avalerio-tkd]

Thanks for this!