Github user nagarajanchinnasamy commented on the issue:
https://github.com/apache/drill/pull/685
@arina-ielchiieva I have made the changes to:
1. Generate sessionId within the constructor of UserSession
2. Add a builder method in UserSession to construct
UserSessionContextInformation
3. Session Parameters related changes are either commented or put in TODO
(for future extension. Will address this in a separate PR)
4. @julianhyde 's comment of being able to invoke session_id() UDF without
parenthesis is still pending. Have sent a mail to dev list. Need to know how to
achieve that in drill.
This time again I have squashed the commits.... as I already have done
that. Will take care of your suggestion about squashing in future PRs.
Please review the changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
