>From Vijay Sarathy <[email protected]>: Attention is currently required from: Ali Alsuliman. Vijay Sarathy has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404 )
Change subject: [ASTERIXDB-3120][ASTERIXDB-3121] CBO Analyze: improve error message. ...................................................................... Patch Set 5: (6 comments) File asterixdb/asterix-common/src/main/java/org/apache/asterix/common/exceptions/ErrorCode.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/e3c93987_979f97f1 PS3, Line 270: INVALID_SAMPLE_SIZE_STRING_OPTIONS(1175), > this should be removed after you revert the change for it. Decided to not remove it, but rename it to INVALID_SAMPLE_SIZE https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/2c4fcbe5_98b1f1a6 PS3, Line 271: INVALID_SAMPLE_SIZE_NUMBER_OPTIONS > change it to INVALID_SAMPLE_SIZE Changed to OUT_OF_RANGE_SAMPLE_SIZE https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/0da0006a_051e5702 PS3, Line 272: INVALID_SAMPLE_SEED_STRING_OPTIONS > change it to INVALID_SAMPLE_SEED Changed to INVALID_SAMPLE_SEED File asterixdb/asterix-common/src/main/resources/asx_errormsg/en.properties: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/e283816a_e1a3d8aa PS3, Line 272: Sample size options are "low", "medium", "high", a positive numeric value, or a string convertible to a positive numeric value > let's change it to: […] Done File asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/statement/AnalyzeStatement.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/16c74b91_48be642a PS3, Line 72: final List<FieldBinding> fbList = options.getFbList(); : for (int i = 0; i < fbList.size(); i++) { > looking at this and usages of ExpressionUtils.toNode(), we should just fix > ExpressionUtils. […] Done. Also added a test case with negative sample-seed https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404/comment/4f5e47eb_18239d51 PS3, Line 142: int v; : try { : v = Integer.parseInt(s); : } catch (NumberFormatException e) { : throw new CompilationException(ErrorCode.INVALID_SAMPLE_SIZE_STRING_OPTIONS); : } : if (!isValidSampleSize(v)) { : throw new CompilationException(ErrorCode.INVALID_SAMPLE_SIZE_NUMBER_OPTIONS, : SAMPLE_LOW_SIZE, SAMPLE_HIGH_SIZE); : } : return v; > we don't need this. […] Done -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17404 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: neo Gerrit-Change-Id: I599005f4f4814f719e422da4ff7d58c2253e5ed4 Gerrit-Change-Number: 17404 Gerrit-PatchSet: 5 Gerrit-Owner: Vijay Sarathy <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Vijay Sarathy <[email protected]> Gerrit-CC: Anon. E. Moose #1000171 Gerrit-Attention: Ali Alsuliman <[email protected]> Gerrit-Comment-Date: Wed, 01 Mar 2023 23:33:09 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Ali Alsuliman <[email protected]> Gerrit-MessageType: comment
