mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jingang Li (JIRA)" <>
Subject [jira] [Created] (MESOS-4416) Get the perf version function return fail
Date Sun, 17 Jan 2016 08:13:39 GMT
Jingang Li created MESOS-4416:

             Summary: Get the perf version function return fail
                 Key: MESOS-4416
             Project: Mesos
          Issue Type: Bug
          Components: c++ api
    Affects Versions: 0.25.0
         Environment: Suse SLE 12 SP1
            Reporter: Jingang Li
            Priority: Minor
             Fix For: 0.26.0

When check the perf version with the "perf::version" gets the command output "perf --version".
The command return string includes a "return key". It cause the version parse fails. If add
the strings::trim, the issue is fixed.

The old code:
Future<Version> version()
  internal::Perf* perf = new internal::Perf({"--version"});
  Future<string> output = perf->output();
  spawn(perf, true);

  return output
    .then([](const string& output) -> Future<Version> {
      // Trim off the leading 'perf version ' text to convert.
      return Version::parse(strings::remove(
          output, "perf version ", strings::PREFIX));

Change to below, issue is fixed:

This message was sent by Atlassian JIRA

View raw message