You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "juntaozhang (via GitHub)" <gi...@apache.org> on 2026/05/28 10:29:49 UTC
[PR] [spark] Align spark.paimon.branch option with explicit branch syntax for chain tables [paimon]
juntaozhang opened a new pull request, #8016:
URL: https://github.com/apache/paimon/pull/8016
### Purpose
#8015
### Tests
SparkChainTableITCase.testChainTableWithBranchOption
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spark] Align spark.paimon.branch option with explicit branch syntax for chain tables [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8016:
URL: https://github.com/apache/paimon/pull/8016#issuecomment-4603803687
Reviewed with focus on alignment between the deprecated `spark.paimon.branch` option and the explicit Java/Paimon `table$branch_xxx` identifier path.
Applying the option before `catalog.getTable(...)` makes the Spark catalog resolve the same branched `Identifier` that explicit branch syntax would use, while leaving an explicit branch name untouched. I also verified the common Spark module compiles with:
```shell
mvn -pl paimon-spark/paimon-spark-common -am -Pfast-build -DskipTests compile
```
No additional blocker from my side.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spark] Align spark.paimon.branch option with explicit branch syntax for chain tables [paimon]
Posted by "juntaozhang (via GitHub)" <gi...@apache.org>.
juntaozhang commented on PR #8016:
URL: https://github.com/apache/paimon/pull/8016#issuecomment-4588847813
Hi @leaves12138, appreciate your feedback, PTAL.
Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spark] Align spark.paimon.branch option with explicit branch syntax for chain tables [paimon]
Posted by "leaves12138 (via GitHub)" <gi...@apache.org>.
leaves12138 commented on PR #8016:
URL: https://github.com/apache/paimon/pull/8016#issuecomment-4586756379
Thanks for the fix. I am holding off on approval for now because the current CI status still has a failing `build_test` job. Please fix or rerun it, then I can take another pass.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spark] Align spark.paimon.branch option with explicit branch syntax for chain tables [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8016:
URL: https://github.com/apache/paimon/pull/8016
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org