Kunal Siyag has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/23857


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, 173 insertions(+), 52 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/23857/1
--
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: newchange
Gerrit-Change-Id: I5e8f2a9784edc86838a375d373e2095dd674d63d
Gerrit-Change-Number: 23857
Gerrit-PatchSet: 1
Gerrit-Owner: Kunal Siyag <[email protected]>

Reply via email to