Zoltan Martonka has posted comments on this change. ( http://gerrit.cloudera.org:8080/24845 )
Change subject: [tserver] Validate deprecated range predicate column type. ...................................................................... Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/24845/1/src/kudu/tserver/tablet_service.cc File src/kudu/tserver/tablet_service.cc: http://gerrit.cloudera.org:8080/#/c/24845/1/src/kudu/tserver/tablet_service.cc@2790 PS1, Line 2790: const int32 > nit: add 'const' to be explicit 'idx' is not going to change? Done http://gerrit.cloudera.org:8080/#/c/24845/1/src/kudu/tserver/tablet_service.cc@2791 PS1, Line 2791: (PREDICT_FALSE(idx == Schema::kC > nit: does it make sense to wrap this into PREDICT_FALSE()? Done http://gerrit.cloudera.org:8080/#/c/24845/1/src/kudu/tserver/tablet_service.cc@2792 PS1, Line 2792: return Status::InvalidArgument( : Substitute("Invalid predicate $0: unknown column.", : SecureShor > nit for here and below: switch to Substitute() (strings::Substitute()) for Done http://gerrit.cloudera.org:8080/#/c/24845/1/src/kudu/tserver/tablet_service.cc@2799 PS1, Line 2799: > nit: this seems to be wrong indent -- should be +2 more spaces for expr con Done -- To view, visit http://gerrit.cloudera.org:8080/24845 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I82ad53ad2263ca587e81fd37d64cc7982060b0f1 Gerrit-Change-Number: 24845 Gerrit-PatchSet: 4 Gerrit-Owner: Zoltan Martonka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Comment-Date: Wed, 16 Sep 2026 08:29:06 +0000 Gerrit-HasComments: Yes
