/**
 * Copyright (C) 2014-2015 LinkedIn Corp. (pinot-core@linkedin.com)
 *
 * 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.linkedin.pinot.tools.admin.command;

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.InputStreamReader;
import java.io.OutputStreamWriter;
import java.net.URL;
import java.net.URLConnection;

import org.json.JSONObject;
import org.kohsuke.args4j.Option;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import com.linkedin.pinot.common.utils.CommonConstants;
import com.linkedin.pinot.common.utils.NetUtil;


public class PostQueryCommand extends AbstractBaseCommand implements Command {
  private static final Logger LOGGER = LoggerFactory.getLogger(PostQueryCommand.class.getName());

  @Option(name = "-brokerHost", required = false, metaVar = "<String>", usage = "host name for controller.")
  private String _brokerHost;

  @Option(name = "-brokerPort", required = false, metaVar = "<int>", usage = "http port for broker.")
  private String _brokerPort = Integer.toString(CommonConstants.Helix.DEFAULT_BROKER_QUERY_PORT);

  @Option(name = "-query", required = true, metaVar = "<string>", usage = "Query string to perform.")
  private String _query;

  @Option(name = "-help", required = false, help = true, aliases = { "-h", "--h", "--help" },
      usage = "Print this message.")
  private boolean _help = false;

  public boolean getHelp() {
    return _help;
  }

  @Override
  public String getName() {
    return "PostQuery";
  }

  @Override
  public String toString() {
    return ("PostQueryCommand -brokerHost " + _brokerHost + " -brokerPort " + _brokerPort + " -query " + _query);
  }

  @Override
  public void cleanup() {

  }

  @Override
  public String description() {
    return "Query the uploaded Pinot segments.";
  }

  public PostQueryCommand setBrokerPort(String port) {
    _brokerPort = port;
    return this;
  }

  public PostQueryCommand setQuery(String query) {
    _query = query;
    return this;
  }

  public String run() throws Exception {
    final JSONObject json = new JSONObject();
    json.put("pql", _query);

    String brokerUrl = "http://" + _brokerHost + ":" + _brokerPort;
    final URLConnection conn = new URL(brokerUrl + "/query").openConnection();
    conn.setDoOutput(true);

    final BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(conn.getOutputStream(), "UTF-8"));
    String request = json.toString();

    writer.write(request, 0, request.length());
    writer.flush();

    final BufferedReader reader = new BufferedReader(new InputStreamReader(conn.getInputStream(), "UTF-8"));
    final StringBuilder sb = new StringBuilder();

    String line = null;
    while ((line = reader.readLine()) != null) {
      sb.append(line);
    }

    return sb.toString();
  }

  public boolean execute() throws Exception {
    if (_brokerHost == null) {
      _brokerHost = NetUtil.getHostAddress();
    }
    LOGGER.info("Executing command: " + toString());
    String result = run();
    LOGGER.info("Result: " + result);
    return true;
  }
}

@@ -2556,24 +2556,160 @@
 Exception %7B%0A
+    if (_brokerHost == null) %7B%0A      _brokerHost = NetUtil.getHostAddress();%0A    %7D%0A    LOGGER.info(%22Executing command: %22 + toString());%0A
     final JS
@@ -3511,144 +3511,8 @@
 n %7B%0A
-    if (_brokerHost == null) %7B%0A      _brokerHost = NetUtil.getHostAddress();%0A    %7D%0A    LOGGER.info(%22Executing command: %22 + toString());%0A
     

Fix for PostQueryCommand in quickstart-offline test