----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40074/ -----------------------------------------------------------
(Updated Nov. 9, 2015, 10:03 a.m.) Review request for MetaModel. Repository: metamodel Description ------- Added validation to check for invalid characters and sheetname length for excel Diffs ----- excel/src/main/java/org/apache/metamodel/excel/ExcelUpdateCallback.java f99ec89 excel/src/test/java/org/apache/metamodel/excel/ExcelUpdateCallbackTest.java 97101bd Diff: https://reviews.apache.org/r/40074/diff/ Testing ------- 1) Added test cases for the same. 2) Verified with name greater than 31 characters. 3) Verified name with invalid characters like [ ] : \ / ? * Thanks, Sai Krishna Kishore Beathanabhotla
