Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

timeout-milliSeconds parameter in ADOMD query is actually seconds #551

Closed
alwortman opened this issue Oct 30, 2019 · 2 comments
Closed

timeout-milliSeconds parameter in ADOMD query is actually seconds #551

alwortman opened this issue Oct 30, 2019 · 2 comments

Comments

@alwortman
Copy link

alwortman commented Oct 30, 2019

When running queries against a SSAS tabular model, the timeout-milliSeconds parameter appears to get translated into seconds incorrectly.

System under test looks like:

<system-under-test>
  <execution>
    <query timeout-milliSeconds="30000" connection-string="@ProvDev"><![CDATA[
        EVALUATE
        FILTER(
          SELECTCOLUMNS (
            SUMMARIZECOLUMNS (
              'headcount_Hiring Manager'[Hiring Manager Name],
              "Measure Name", "Historical Total Open Requisitions",
              "Value", [Historical Total Open Requisitions]
            ),
            "Dimension Name", "'headcount_Hiring Manager'[Hiring Manager Name]",
            "Dimension Value", 'headcount_Hiring Manager'[Hiring Manager Name],
            "Measure Name", "Historical Total Open Requisitions",
            "Measure Value", [Historical Total Open Requisitions]
          ),
          [Dimension Value] <> ""
        )
        ]]></query>
  </execution>
</system-under-test>

I cloned the develop branch and ran the ExecuteTestCases unit test. When the query executed, the timeout argument was in seconds instead of milliseconds.

image

The workaround is obviously to just lower the timeout to your desired threshold in seconds, but it is confusing as-is.

Let me know if you need any additional info.

@Seddryck Seddryck added the bug label Oct 31, 2019
@Seddryck
Copy link
Owner

Thank you very much for the report.

I will fix that during the next days (or feel free to submit a PR) and it will be the final fix before v1.21 release.

@Seddryck Seddryck added this to the v1.21 milestone Nov 1, 2019
@Seddryck
Copy link
Owner

Seddryck commented Nov 1, 2019

Beta available at https://ci.appveyor.com/project/Seddryck/nbi/builds/28543510/artifacts or 1.22.0-beta0013 will also be included in v1.21

@Seddryck Seddryck closed this as completed Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants