hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phabricator (JIRA)" <>
Subject [jira] [Commented] (HIVE-2804) Task log retrieval fails on secure cluster
Date Sun, 06 May 2012 21:50:49 GMT


Phabricator commented on HIVE-2804:

cwsteinbach has requested changes to the revision "HIVE-2804 [jira] Task log retrieval fails
on secure cluster".

  ql/src/java/org/apache/hadoop/hive/ql/exec/ This doesn't belong
here. The UDF is for testing purposes only. Users should not see it listed in the output of
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/ @Namit: Good point.

  @Zhenxiao: Please put this in ql/src/test/org/apache/hadoop/hive/ql/udf/generic, and then
take a look at ql/src/test/queries/clientpositive/create_genericudf.q for an example of how
to register a temporary UDF.
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/ Might be good to
change the name to "evaluate_npe" (and update the other comments accordingly) just to make
it clear that the NPE is thrown in evaluate() as opposed to initialize().
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/ I'm curious if this
if() block is really necessary. Does the Java compiler complain without it?
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/ Is it possible to
write a GenericUDF that takes no input parameters (e.g. like UDFPI)? If so then I think we
should do that here since we ignore the input anyway. If that isn't possible, then please
change this to take a string as input since that will work better with the src table.
  ql/src/test/queries/clientnegative/cluster_npe_tasklog.q:3 Referencing src_thrift may give
people the impression that this test is somehow related to Thrift. Let's use the src table
  ql/src/test/queries/clientnegative/cluster_npe_tasklog.q:1 Please change the name to "cluster_tasklog_retrieval.q".
  shims/src/0.20/java/org/apache/hadoop/hive/shims/ Can we call TaskLogServlet.getTaskLogUrl()
here instead of manually constructing the URL? If the answer is no then please add a comment
explaining why. Thanks.
  shims/src/0.20S/java/org/apache/hadoop/hive/shims/ Same question as
  shims/src/0.23/java/org/apache/hadoop/hive/shims/ Please use the getHost()
and getPort() methods that are provided by



> Task log retrieval fails on secure cluster
> ------------------------------------------
>                 Key: HIVE-2804
>                 URL:
>             Project: Hive
>          Issue Type: Bug
>          Components: Diagnosability, Query Processor, Security
>            Reporter: Carl Steinbach
>            Assignee: Zhenxiao Luo
>         Attachments: HIVE-2804.1.patch.txt, HIVE-2804.D3057.1.patch

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message