kumaab commented on code in PR #307: URL: https://github.com/apache/ranger/pull/307#discussion_r1548609407
########## ranger-tools/src/main/python/stress/stress-hbase-loadgenerator.py: ########## @@ -0,0 +1,106 @@ +import subprocess +import time +import argparse +import os +from datetime import datetime + +def increase_memory_for_loadgenerator(): + try: + cmd = "export HBASE_OPTS='-Xmx10g'" + print(cmd) + op = subprocess.call(cmd, shell=True) + print("Output:", op) + except subprocess.CalledProcessError as e: + print("Error in setting HBASE_HEAPSIZE:", e) + exit(1) +def login(): + try: + cmd = "kinit -kt <systest.keytab> systest" Review Comment: Instead of hardcoding the value it is better to provide the keytab as an argument. Alternatively, doing the kinit externally and documenting the steps in readme also works. -- 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: dev-unsubscr...@ranger.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org