/*
 * 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.tests;

import com.facebook.presto.Session;
import com.facebook.presto.SystemSessionProperties;
import com.facebook.presto.tpch.TpchPlugin;
import com.google.common.collect.ImmutableMap;
import org.testng.annotations.Test;

import java.nio.file.Paths;

import static com.facebook.presto.testing.TestingSession.testSessionBuilder;
import static com.facebook.presto.tpch.TpchMetadata.TINY_SCHEMA_NAME;

public class TestDistributedSpilledQueries
        extends AbstractTestQueries
{
    public TestDistributedSpilledQueries()
    {
        this(TestDistributedSpilledQueries::createQueryRunner);
    }

    protected TestDistributedSpilledQueries(QueryRunnerSupplier queryRunnerSupplier)
    {
        super(queryRunnerSupplier);
    }

    public static DistributedQueryRunner createQueryRunner()
            throws Exception
    {
        Session defaultSession = testSessionBuilder()
                .setCatalog("tpch")
                .setSchema(TINY_SCHEMA_NAME)
                .setSystemProperty(SystemSessionProperties.TASK_CONCURRENCY, "2")
                .setSystemProperty(SystemSessionProperties.SPILL_ENABLED, "true")
                .setSystemProperty(SystemSessionProperties.JOIN_SPILL_ENABLED, "true")
                .setSystemProperty(SystemSessionProperties.AGGREGATION_OPERATOR_UNSPILL_MEMORY_LIMIT, "128kB")
                .setSystemProperty(SystemSessionProperties.USE_MARK_DISTINCT, "false")
                .build();

        ImmutableMap<String, String> extraProperties = ImmutableMap.<String, String>builder()
                .put("experimental.spill-enabled", "true")
                .put("experimental.spiller-spill-path", Paths.get(System.getProperty("java.io.tmpdir"), "presto", "spills").toString())
                .put("experimental.spiller-max-used-space-threshold", "1.0")
                .put("experimental.memory-revoking-threshold", "0.0") // revoke always
                .put("experimental.memory-revoking-target", "0.0")
                .build();

        DistributedQueryRunner queryRunner = new DistributedQueryRunner(defaultSession, 2, extraProperties);

        try {
            queryRunner.installPlugin(new TpchPlugin());
            queryRunner.createCatalog("tpch", "tpch");
            return queryRunner;
        }
        catch (Exception e) {
            queryRunner.close();
            throw e;
        }
    }

    @Test(enabled = false)
    public void testJoinPredicatePushdown()
    {
        // TODO: disabled until join spilling is reworked
    }

    @Test(enabled = false)
    @Override
    public void testAssignUniqueId()
    {
        // TODO: disabled until https://github.com/prestodb/presto/issues/8926 is resolved
        //       due to long running query test created many spill files on disk.
    }

    @Test(enabled = false)
    public void testLimitWithJoin()
    {
        // TODO: disable until https://github.com/prestodb/presto/issues/13859 is resolved.
    }

    @Test(enabled = false)
    public void testJoinDoubleClauseWithRightOverlap()
    {
        // TODO: disable until https://github.com/prestodb/presto/issues/13859 is resolved.
    }

    @Test(enabled = false)
    @Override
    public void testCorrelatedNonAggregationScalarSubqueries()
    {
        // TODO: disable until https://github.com/prestodb/presto/issues/15542 is resolved
    }
}

@@ -2956,150 +2956,8 @@
  %7D%0A%0A
-    @Test(enabled = false)%0A    public void testJoinPredicatePushdown()%0A    %7B%0A        // TODO: disabled until join spilling is reworked%0A    %7D%0A%0A
     
@@ -3220,363 +3220,8 @@
  %7D%0A%0A
-    @Test(enabled = false)%0A    public void testLimitWithJoin()%0A    %7B%0A        // TODO: disable until https://github.com/prestodb/presto/issues/13859 is resolved.%0A    %7D%0A%0A    @Test(enabled = false)%0A    public void testJoinDoubleClauseWithRightOverlap()%0A    %7B%0A        // TODO: disable until https://github.com/prestodb/presto/issues/13859 is resolved.%0A    %7D%0A%0A
     

Remove test "overrides" that are no longer relevant