/*
 * Licensed to CRATE Technology GmbH ("Crate") under one or more contributor
 * license agreements.  See the NOTICE file distributed with this work for
 * additional information regarding copyright ownership.  Crate 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.
 *
 * However, if you have executed another commercial license agreement
 * with Crate these terms will supersede the license and you may use the
 * software solely pursuant to the terms of the relevant commercial agreement.
 */

package io.crate.analyze;

import io.crate.data.Row;
import io.crate.sql.tree.AstVisitor;
import io.crate.sql.tree.Node;
import io.crate.sql.tree.ParameterExpression;
import io.crate.sql.tree.StringLiteral;
import org.apache.lucene.util.BytesRef;

import javax.annotation.Nullable;
import java.util.Locale;

public class SafeExpressionToStringVisitor extends AstVisitor<String, Row> {

    private static final SafeExpressionToStringVisitor INSTANCE = new SafeExpressionToStringVisitor();

    private SafeExpressionToStringVisitor() {
    }

    public static String convert(Node node, @Nullable Row context) {
        return INSTANCE.process(node, context);
    }

    @Override
    protected String visitStringLiteral(StringLiteral node, Row parameters) {
        return node.getValue();
    }

    @Override
    public String visitParameterExpression(ParameterExpression node, Row parameters) {
        Object value = parameters.get(node.index());
        if (value instanceof BytesRef) {
            return ((BytesRef) value).utf8ToString();
        }
        if (!(value instanceof String)) {
            throw new IllegalArgumentException(String.format(Locale.ENGLISH, "Parameter %s not a string value, can't handle this.", value));
        }
        return (String) value;
    }

    @Override
    protected String visitNode(Node node, Row context) {
        throw new IllegalArgumentException(String.format(Locale.ENGLISH, "Can't handle %s.", node));
    }
}

@@ -1106,16 +1106,53 @@
 isitor;%0A
+import io.crate.sql.tree.Expression;%0A
 import i
@@ -1305,42 +1305,8 @@
 f;%0A%0A
-import javax.annotation.Nullable;%0A
 impo
@@ -1334,16 +1334,22 @@
 %0Apublic 
+final 
 class Sa
@@ -1605,40 +1605,45 @@
 ert(
-Node node, @Nullable Row context
+Expression expression, Row parameters
 ) %7B%0A
@@ -1678,21 +1678,30 @@
 ess(
-node, context
+expression, parameters
 );%0A 
@@ -1774,20 +1774,23 @@
 Literal 
-node
+literal
 , Row pa
@@ -1816,20 +1816,23 @@
  return 
-node
+literal
 .getValu
@@ -1920,20 +1920,29 @@
 ression 
-node
+parameterExpr
 , Row pa
@@ -1991,20 +1991,29 @@
 ers.get(
-node
+parameterExpr
 .index()
@@ -2457,32 +2457,45 @@
 gumentException(
+%0A            
 String.format(Lo
@@ -2509,16 +2509,68 @@
 GLISH, %22
+Expected a String literal or a ParameterExpression. 
 Can't ha

Remove invalid nullable annotation from SafeExpressionToStringVisitor