/*
 * Licensed 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 com.facebook.presto.metadata;

import com.facebook.presto.spi.ColumnHandle;
import com.facebook.presto.spi.predicate.TupleDomain;
import com.facebook.presto.sql.planner.plan.TableScanNode;

import java.util.List;

import static com.google.common.collect.ImmutableList.toImmutableList;

public class TableLayoutResult
{
    private final TableLayout layout;
    private final TupleDomain<ColumnHandle> unenforcedConstraint;

    public TableLayoutResult(TableLayout layout, TupleDomain<ColumnHandle> unenforcedConstraint)
    {
        this.layout = layout;
        this.unenforcedConstraint = unenforcedConstraint;
    }

    public TableLayout getLayout()
    {
        return layout;
    }

    public TupleDomain<ColumnHandle> getUnenforcedConstraint()
    {
        return unenforcedConstraint;
    }

    public boolean hasAllOutputs(TableScanNode node)
    {
        List<ColumnHandle> nodeColumnHandles = node.getOutputSymbols().stream()
                .map(node.getAssignments()::get)
                .collect(toImmutableList());
        return getLayout().getColumns()
                .map(columnHandles -> columnHandles.containsAll(nodeColumnHandles))
                .orElse(true);
    }
}

@@ -756,29 +756,98 @@
 de;%0A
-%0Aimport java.util.Lis
+import com.google.common.collect.ImmutableSet;%0A%0Aimport java.util.List;%0Aimport java.util.Se
 t;%0A%0A
@@ -1489,32 +1489,199 @@
 ode node)%0A    %7B%0A
+        if (!layout.getColumns().isPresent()) %7B%0A            return true;%0A        %7D%0A        Set%3CColumnHandle%3E columns = ImmutableSet.copyOf(layout.getColumns().get());%0A
         List%3CCol
@@ -1834,24 +1834,25 @@
 bleList());%0A
+%0A
         retu
@@ -1858,145 +1858,45 @@
 urn 
-getLayout().getColumns()%0A                .map(columnHandles -%3E columnHandles.containsAll(nodeColumnHandles))%0A                .orElse(true
+columns.containsAll(nodeColumnHandles
 );%0A 

Refactor TableLayoutResult#hasAllOutputs