[ 
https://issues.apache.org/jira/browse/DRILL-4280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15850925#comment-15850925
 ] 

ASF GitHub Bot commented on DRILL-4280:
---------------------------------------

Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/578#discussion_r99257539
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/rpc/data/DataConnectionCreator.java
 ---
    @@ -37,40 +37,28 @@
       static final org.slf4j.Logger logger = 
org.slf4j.LoggerFactory.getLogger(DataConnectionCreator.class);
     
       private volatile DataServer server;
    -  private final BootStrapContext context;
    -  private final WorkEventBus workBus;
    -  private final WorkerBee bee;
    -  private final boolean allowPortHunting;
    -  private ConcurrentMap<DrillbitEndpoint, DataConnectionManager> 
connectionManager = Maps.newConcurrentMap();
    -  private final BufferAllocator dataAllocator;
    +  private final ConcurrentMap<DrillbitEndpoint, DataConnectionManager> 
connectionManager = Maps.newConcurrentMap();
     
    -  public DataConnectionCreator(
    -      BootStrapContext context,
    -      BufferAllocator allocator,
    -      WorkEventBus workBus,
    -      WorkerBee bee,
    -      boolean allowPortHunting) {
    -    super();
    -    this.context = context;
    -    this.workBus = workBus;
    -    this.bee = bee;
    -    this.allowPortHunting = allowPortHunting;
    -    this.dataAllocator = allocator;
    +  private final BitConnectionConfigImpl config;
    +
    +  public DataConnectionCreator(BootStrapContext context, BufferAllocator 
allocator, WorkEventBus workBus,
    +                               WorkerBee bee) {
    +    config = new BitConnectionConfigImpl(allocator, context, new 
DataServerRequestHandler(workBus, bee));
       }
     
    -  public DrillbitEndpoint start(DrillbitEndpoint partialEndpoint) throws 
DrillbitStartupException {
    -    server = new DataServer(context, dataAllocator, workBus, bee);
    +  public DrillbitEndpoint start(DrillbitEndpoint partialEndpoint, boolean 
allowPortHunting)
    +      throws DrillbitStartupException {
    --- End diff --
    
    Same as ControlServer


> Kerberos Authentication
> -----------------------
>
>                 Key: DRILL-4280
>                 URL: https://issues.apache.org/jira/browse/DRILL-4280
>             Project: Apache Drill
>          Issue Type: Improvement
>            Reporter: Keys Botzum
>            Assignee: Sudheesh Katkam
>              Labels: security
>
> Drill should support Kerberos based authentication from clients. This means 
> that both the ODBC and JDBC drivers as well as the web/REST interfaces should 
> support inbound Kerberos. For Web this would most likely be SPNEGO while for 
> ODBC and JDBC this will be more generic Kerberos.
> Since Hive and much of Hadoop supports Kerberos there is a potential for a 
> lot of reuse of ideas if not implementation.
> Note that this is related to but not the same as 
> https://issues.apache.org/jira/browse/DRILL-3584 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to