Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HadoopSecureHiveWrapper - CliSessionState - connection should be closed #255

Open
kardoo opened this issue Jan 6, 2017 · 1 comment
Open

Comments

@kardoo
Copy link

kardoo commented Jan 6, 2017

Hive session is not being closed. This is not a problem when using mapreduce.
When using tez, AM sessions keep running until tez timeout expires.

I suggest to change HadoopSecureHiveWrapper.java :

	int returnCode = 0;
	try {
		returnCode = cli.processFile(hiveScript);
	} catch (Exception ex) {
		logger.error("Error processing file: ", ex);
	} finally {
		logger.info("Closing session state.");
		if (ss != null) ss.close();
	}
	if (returnCode != 0) {
		logger.warn("Got exception " + returnCode + " from line: " + hiveScript);
		throw new HiveQueryExecutionException(returnCode, hiveScript);
	}
@kardoo
Copy link
Author

kardoo commented Jan 9, 2017

I opened a Pull Request:

@kardoo kardoo changed the title HadoopSecureHiveWrapper - CliSessionState close HadoopSecureHiveWrapper - CliSessionState - connection should be closed Mar 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant