[
https://issues.apache.org/jira/browse/HIVE-22329?focusedWorklogId=329951&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-329951
]
ASF GitHub Bot logged work on HIVE-22329:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 17/Oct/19 15:36
Start Date: 17/Oct/19 15:36
Worklog Time Spent: 10m
Work Description: kgyrtkirk commented on pull request #810: HIVE-22329
Create DriverContext
URL: https://github.com/apache/hive/pull/810#discussion_r336079338
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/lock/LockTableOperation.java
##########
@@ -34,7 +34,7 @@ public LockTableOperation(DDLOperationContext context,
LockTableDesc desc) {
@Override
public int execute() throws HiveException {
- Context ctx = context.getDriverContext().getCtx();
+ Context ctx = context.getDriverQueue().getCtx();
Review comment:
apparently this is not a queue: why would a queue would have `getCtx()`
method
----------------------------------------------------------------
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:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 329951)
Time Spent: 20m (was: 10m)
> Create DriverContext
> --------------------
>
> Key: HIVE-22329
> URL: https://issues.apache.org/jira/browse/HIVE-22329
> Project: Hive
> Issue Type: Sub-task
> Reporter: Miklos Gergely
> Assignee: Miklos Gergely
> Priority: Major
> Labels: pull-request-available
> Fix For: 4.0.0
>
> Attachments: HIVE-22329.01.patch, HIVE-22329.02.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> The current DriverContext class is not really a context, rather a queue, thus
> it should be renamed to DriverQueue. At the same time Driver has a a lot of
> fields, they should be in a real context class called DriverContext, which
> would also facilitate the extraction of independent parts of the Driver class.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
