nsivabalan commented on code in PR #17940:
URL: https://github.com/apache/hudi/pull/17940#discussion_r2738537672


##########
hudi-cli/src/main/java/org/apache/hudi/cli/commands/MetadataCommand.java:
##########
@@ -387,6 +392,44 @@ public String validateFiles(
     return HoodiePrintHelper.print(header, new HashMap<>(), "", false, 
Integer.MAX_VALUE, false, rows);
   }
 
+  // @ShellOption(value = "--backup", help = "Backup the metadata table before 
delete", defaultValue = "true", arity = 1) final boolean backup
+  @ShellMethod(key = "metadata lookup-record-index", value = "Print Record 
index information for a record_key")

Review Comment:
   should we support a list of record keys. comma separated. 
   Also, we have partitioned RLI support in latest 1.x. 
   We might need to take in partition path as input in such case. 
   
   even if we don't have bandwidth to support non global variant, lets check 
for RLI index definition and if its non global variant, we should throw 
exception for now. 
   
   For users who are looking to read records from MDT global RLI index, it 
should succeed.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to