Bodor Laszlo

Results 133 comments of Bodor Laszlo

I can see that Zoltan's repo contains an entrypoint script: https://github.com/kgyrtkirk/hive-test-kube/blob/master/htk-jenkins/entrypoint isn't that missing from here? I guess we want to track it together with the Dockerfile thanks for this...

> yes - it was a surprise to me as well that k8s doesn't support `VOLUME`-s; the default config copy was a courtesy to make it usable right away if...

thanks for this patch @mszurap this patch makes sense to me, only 1 note regarding: ``` sslTrustStore=/path/to/mydir with spaces/truststore.jks ``` even if I hate to see spaces in folder names,...

while testing this patch, I realized that the meta hook commit process doesn't respect hiveLockEnabled, modified it, quite a small change but not related query history, please let me know...

> IMO we should split it out into a separate ticket, if it isn't related to QueryHistory ack, just created https://github.com/apache/hive/pull/5687

unit test passed without the locking hack (most probably it was an incomplete step that I worked around with proper configuration in the meantime) also addressed sonarqube issues

the generic metastore service logic has been moved to a separate jira/PR: https://github.com/apache/hive/pull/5786 this PR remains idle until we can merge that one, and here, only query_history related changes will...

> @abstractdog, can we close this PR as a generic solution was already merged? let me check if a minimal change is needed here to make the history table handled,...

@deniskuzZ , @ayushtkn : final patch here is as simple as https://github.com/apache/hive/pull/5666/commits/6b627a1f2dee9f478ba45d60aac8a2fcafbf704c could this go to 4.1.0?

opening new PR to check precommits already known failure to be checked: TestHplSqlViaBeeLine