impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Knupp (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-2013: Issue Hbase queries individually during data-load.
Date Fri, 22 Jul 2016 22:44:46 GMT
David Knupp has uploaded a new patch set (#5).

Change subject: IMPALA-2013: Issue Hbase queries individually during data-load.

IMPALA-2013: Issue Hbase queries individually during data-load.

Loading data into HBase has traditionally been a bit flaky, with
problems being hard to diagnose from existing logs. I think this is
at least in part due to the fact that we have been relying on a
command file to send queries to the HBase shell. When sending a
series of queries in a file, the HBase shell does not check or
halt operation after each query.


"There is no way to programmatically check each individual command for
success or failure. Also, though you see the output for each command,
the commands themselves are not echoed to the screen so it can be
difficult to line up the command with its output."

Even if the HBase process dies completely, our data load process
goes through the laborious process of continuing to send commands
to the shell.

Instead of trying to process the file all at once, the command file
generated by should be iterated
line-by-line, with each query being passed individually to the HBase
shell, checking for errors in the output each time. If we get an
error message, fail fast and loudly.

Also, this commit fixes several flake8 linter complaints, and replaces
print statements with specific log level output.

Change-Id: I911d972ba8ad3a2a084c8195074556153722c7e2
M bin/
1 file changed, 139 insertions(+), 61 deletions(-)

  git pull ssh:// refs/changes/28/3728/5
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I911d972ba8ad3a2a084c8195074556153722c7e2
Gerrit-PatchSet: 5
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: David Knupp <>
Gerrit-Reviewer: David Knupp <>
Gerrit-Reviewer: Harrison Sheinblatt <>
Gerrit-Reviewer: Ishaan Joshi <>
Gerrit-Reviewer: Michael Brown <>

View raw message