/*
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
 * with the License.  You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.drill.exec.store.mapr.db;

import java.util.Iterator;
import java.util.List;

import com.fasterxml.jackson.annotation.JsonIgnore;
import org.apache.drill.common.exceptions.ExecutionSetupException;
import org.apache.drill.common.expression.SchemaPath;
import org.apache.drill.common.logical.StoragePluginConfig;
import org.apache.drill.exec.physical.base.AbstractBase;
import org.apache.drill.exec.physical.base.PhysicalOperator;
import org.apache.drill.exec.physical.base.PhysicalVisitor;
import org.apache.drill.exec.physical.base.SubScan;
import org.apache.drill.exec.store.StoragePluginRegistry;

import com.fasterxml.jackson.annotation.JacksonInject;
import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.annotation.JsonTypeName;
import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableSet;

// Class containing information for reading a single HBase region
@JsonTypeName("maprdb-sub-scan")
public class MapRDBSubScan extends AbstractBase implements SubScan {
  private static final org.slf4j.Logger logger = org.slf4j.LoggerFactory.getLogger(MapRDBSubScan.class);

  private final MapRDBFormatPlugin formatPlugin;
  private final List<MapRDBSubScanSpec> regionScanSpecList;
  private final List<SchemaPath> columns;
  private final String tableType;

  @JsonCreator
  public MapRDBSubScan(@JacksonInject StoragePluginRegistry engineRegistry,
                       @JsonProperty("userName") String userName,
                       @JsonProperty("formatPluginConfig") MapRDBFormatPluginConfig formatPluginConfig,
                       @JsonProperty("storageConfig") StoragePluginConfig storageConfig,
                       @JsonProperty("regionScanSpecList") List<MapRDBSubScanSpec> regionScanSpecList,
                       @JsonProperty("columns") List<SchemaPath> columns,
                       @JsonProperty("tableType") String tableType) throws ExecutionSetupException {
    this(userName,
        (MapRDBFormatPlugin) engineRegistry.getFormatPlugin(storageConfig, formatPluginConfig),
        regionScanSpecList,
        columns,
        tableType);
  }

  public MapRDBSubScan(String userName, MapRDBFormatPlugin formatPlugin,
      List<MapRDBSubScanSpec> maprSubScanSpecs, List<SchemaPath> columns, String tableType) {
    super(userName);
    this.formatPlugin = formatPlugin;
    this.regionScanSpecList = maprSubScanSpecs;
    this.columns = columns;
    this.tableType = tableType;
  }

  public List<MapRDBSubScanSpec> getRegionScanSpecList() {
    return regionScanSpecList;
  }

  public List<SchemaPath> getColumns() {
    return columns;
  }

  @Override
  public boolean isExecutable() {
    return false;
  }

  @Override
  public <T, X, E extends Throwable> T accept(PhysicalVisitor<T, X, E> physicalVisitor, X value) throws E {
    return physicalVisitor.visitSubScan(this, value);
  }

  @Override
  public PhysicalOperator getNewWithChildren(List<PhysicalOperator> children) {
    Preconditions.checkArgument(children.isEmpty());
    return new MapRDBSubScan(getUserName(), formatPlugin, regionScanSpecList, columns, tableType);
  }

  @Override
  public Iterator<PhysicalOperator> iterator() {
    return ImmutableSet.<PhysicalOperator>of().iterator();
  }

  @Override
  public int getOperatorType() {
    return 1001;
  }

  public String getTableType() {
    return tableType;
  }

  @JsonIgnore
  public MapRDBFormatPlugin getFormatPlugin() {
    return formatPlugin;
  }

  @JsonIgnore
  public MapRDBFormatPluginConfig getFormatPluginConfig() {
    return (MapRDBFormatPluginConfig) formatPlugin.getConfig();
  }

}

@@ -3331,24 +3331,361 @@
 eType;%0A  %7D%0A%0A
+%0A  @JsonProperty(%22formatPluginConfig%22)%0A  public MapRDBFormatPluginConfig getFormatPluginConfig() %7B%0A    return (MapRDBFormatPluginConfig) formatPlugin.getConfig();%0A  %7D%0A%0A  @JsonProperty(%22storageConfig%22)%0A  public StoragePluginConfig getStorageConfig()%7B%0A    return formatPlugin.getStorageConfig();%0A  %7D%0A%0A  @JsonProperty(%22regionScanSpecList%22)%0A
   public Lis
@@ -3763,24 +3763,51 @@
 cList;%0A  %7D%0A%0A
+  @JsonProperty(%22columns%22)%0A
   public Lis
@@ -3831,24 +3831,24 @@
 Columns() %7B%0A
-
     return c
@@ -3856,24 +3856,113 @@
 lumns;%0A  %7D%0A%0A
+  @JsonProperty(%22tableType%22)%0A  public String getTableType() %7B%0A    return tableType;%0A  %7D%0A%0A
   @Override%0A
@@ -4642,68 +4642,8 @@
  %7D%0A%0A
-  public String getTableType() %7B%0A    return tableType;%0A  %7D%0A%0A
   @J
@@ -4734,149 +4734,6 @@
  %7D%0A%0A
-  @JsonIgnore%0A  public MapRDBFormatPluginConfig getFormatPluginConfig() %7B%0A    return (MapRDBFormatPluginConfig) formatPlugin.getConfig();%0A  %7D%0A%0A
 %7D%0A

Fix NPE during MapRDBSubScan deserialization