[ https://issues.apache.org/jira/browse/HIVE-862?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Zheng Shao updated HIVE-862: ---------------------------- Resolution: Fixed Fix Version/s: 0.6.0 Release Note: HIVE-862. CommandProcessor should return DriverResponse. (Bill Graham via zshao) (was: Refactored CommandProcessor interface and it's implementations to return CommandProcessorResponse object, instead of an int response code. This is a non-backward compatible change, but one that's necessary to better message clients.) Hadoop Flags: [Reviewed] Status: Resolved (was: Patch Available) Committed. Thanks Bill! > CommandProcessor should return DriverResponse > --------------------------------------------- > > Key: HIVE-862 > URL: https://issues.apache.org/jira/browse/HIVE-862 > Project: Hadoop Hive > Issue Type: New Feature > Components: Query Processor > Reporter: Edward Capriolo > Assignee: Bill Graham > Fix For: 0.6.0 > > Attachments: HIVE-862.1.patch, HIVE-862.2.patch > > > Since Driver now returns DriverResponse CommandProcessor should return a > similar object. > {noformat} > For reference here is how i want to use this: > Currently: > HWISessionItem.java > private List<Integer> queryRet; > private List<String> queries; > public void runQuery() { > //for each query in queries > if (instanceof driver) { > //other stuff here > queryRet.add( qp.run(cmd) ); > } else { > queryRet.add(proc.run(cmd_1) ); > } > } > So I would like to refactor and use DriverResponse. > private List<DriverResponse> queryRet; > private List<String> queries; > public void runQuery() { > //for each query in queries > > if (instanceof driver) { > //other stuff here > queryRet.add( qp.run(cmd) ); //<--- this is fine > } else { > queryRet.add(proc.run(cmd_1) ); <--this returns an int. > } > } > {noformat} -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.