Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23857
to look at the new patch set (#2).
Change subject: IMPALA-12918: Validate numeric values for table stats properties
......................................................................
IMPALA-12918: Validate numeric values for table stats properties
This patch adds validation for the numRows, totalSize, and rawDataSize
table properties during ALTER TABLE SET TBLPROPERTIES operations.
Previously, these properties could be set to empty strings or non-numeric
values, which could cause NumberFormatException errors in downstream
applications like Spark when they try to parse these values.
The fix adds a new method analyzeTableStatsProperties() that validates:
- The value is not null or empty
- The value is parseable as a long
Testing:
- Added new test class TableStatsValidationBugTest with tests for
empty values, non-numeric values, and valid numeric values
Change-Id: I5e8f2a9784edc86838a375d373e2095dd674d63d
---
M fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java
A fe/src/test/java/org/apache/impala/analysis/TableStatsValidationBugTest.java
2 files changed, 175 insertions(+), 53 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/23857/2
--
To view, visit http://gerrit.cloudera.org:8080/23857
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5e8f2a9784edc86838a375d373e2095dd674d63d
Gerrit-Change-Number: 23857
Gerrit-PatchSet: 2
Gerrit-Owner: Kunal Siyag <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
