HIVE-26126: Allow capturing/validating SQL generated from HMS calls i…
What changes were proposed in this pull request?
- Create logger configuration (tests only) for writing specific DataNucleus SQL queries to (new) operation log files.
- Add hook copying the content of datanucleus log files to session's console for use in qtests.
- Avoid creating appenders & files when hook is inactive.
- Enable the hook on certain tests with partitions and update output.
Why are the changes needed?
For motivation see HIVE-26126
Does this PR introduce any user-facing change?
No, the change only affects test.
How was this patch tested?
See changes in .q and .q.out files
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.
Hey bot please keep this open!
+1, changes look good to me!
Thanks for the review @dengzhhu653 ! My main concern at the moment is that the output depends on the state of the metastore. Since we are reusing the derby database across tests the output depends on the number and order that tests are executed. I cannot merge this till we find a solution where the output is stable and predictable.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.