You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "leaves12138 (via GitHub)" <gi...@apache.org> on 2026/05/28 10:00:43 UTC
[PR] [core] Return null for blob view referencing null value [paimon]
leaves12138 opened a new pull request, #8013:
URL: https://github.com/apache/paimon/pull/8013
### Purpose
When a blob view references an upstream row whose blob value is SQL null, the reader should faithfully return null instead of treating the reference as missing.
### Changes
- Track preloaded blob view references as descriptor values or null values.
- Make `BlobViewResolvingRow` report `isNullAt` and `getBlob` consistently for blob views resolved to null.
- Keep missing references fail-fast by distinguishing them from references that resolve to null.
- Extend `BlobTableTest#testBlobViewE2E` to cover upstream null blob values.
### Tests
- `mvn -pl paimon-common,paimon-core -DskipTests compile`
- `mvn -pl paimon-common,paimon-core spotless:check -DskipTests`
- `mvn -pl paimon-core -am -Pfast-build -DfailIfNoTests=false -Dtest=BlobTableTest#testBlobViewE2E+testBlobViewResolutionFailsOnMissingUpstream test`
--
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] [core] Return null for blob view referencing null value [paimon]
Posted by "leaves12138 (via GitHub)" <gi...@apache.org>.
leaves12138 commented on PR #8013:
URL: https://github.com/apache/paimon/pull/8013#issuecomment-4564506043
LGTM from my side.
I rechecked the null-upstream blob view flow. The change now distinguishes upstream null values from missing references, and `isNullAt`/`getBlob` expose the resolved null consistently. I also ran the focused `BlobTableTest` cases for null upstream, forwarded `BlobView` references, and missing upstream failure; all passed.
--
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] [core] Return null for blob view referencing null value [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8013:
URL: https://github.com/apache/paimon/pull/8013#issuecomment-4564507560
+1
--
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] [core] Return null for blob view referencing null value [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8013:
URL: https://github.com/apache/paimon/pull/8013
--
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