You are viewing a plain text version of this content. The canonical link for it is here.
Posted to codereview@trafodion.apache.org by GitBox <gi...@apache.org> on 2019/10/01 14:06:15 UTC
[GitHub] [trafodion] selvaganesang opened a new pull request #1858:
[TRAFODION-3328] Code cleanup in NADefaults
selvaganesang opened a new pull request #1858: [TRAFODION-3328] Code cleanup in NADefaults
URL: https://github.com/apache/trafodion/pull/1858
Avoid reading VERSIONS and DEFAULTS table after it is done once in a process.
This required following cleanup
1. cqd MODE_SEABASE is removed.
2. Removed the support to read CQD from a file.
3. NADefaults constructor now reads the defaults table once and
saves off the relevant information such as currentDefaults, currentTokens,
provenances and currentFloats to be reused by the other NADefaults objects.
4. T2 driver doesn't use the defaultContext to compile any query. Hence, it is not
possible to use the heap from CmpContext to save off the defaults value.
The code has been changed the heap from CliGlobals to save the current defaults.
5. Fixed code to use static literals as it is from the defaultsDefaults to
avoid invalid memory in the saved defaults
6. Changes to optimize initialization of currentTokens_ in NADefaults
7. CQD * RESET RESET
CQD * RESET
Now resets to the defaults when CQD * RESET RESET was issued.
8. Also optimized initCurrentsFromDefaultsTable further.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services