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

Add missing quotes to big5 Jinja template #312

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

gkamat
Copy link
Collaborator

@gkamat gkamat commented Jun 10, 2024

Description

Minor fix to big5 workload; missing quotes were causing subsequent corpus sizes in workload.json to raise errors.

Testing

Verified that all corpus sizes available are accepted and do not raise errors.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@gkamat gkamat added backport 2 Backport to the "2" branch backport 1 backport 3 Backport to the "3" branch backport 7 Backport to the "7" branch labels Jun 10, 2024
@@ -35,7 +35,7 @@
"compressed-bytes": 53220934846,
"uncompressed-bytes": 943679382267
}
{% elif corpus_size == 1000-full %}
{% elif corpus_size == "1000-full" %}
{
"source-file": "documents-1000-full.json.bz2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we also need to include this in files.txt?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@IanHoang IanHoang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments

@gkamat gkamat merged commit 411e304 into opensearch-project:main Jun 10, 2024
6 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 10, 2024
Signed-off-by: Govind Kamat <[email protected]>
(cherry picked from commit 411e304)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 10, 2024
Signed-off-by: Govind Kamat <[email protected]>
(cherry picked from commit 411e304)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 10, 2024
Signed-off-by: Govind Kamat <[email protected]>
(cherry picked from commit 411e304)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 10, 2024
Signed-off-by: Govind Kamat <[email protected]>
(cherry picked from commit 411e304)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
gkamat pushed a commit that referenced this pull request Jun 10, 2024
(cherry picked from commit 411e304)

Signed-off-by: Govind Kamat <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
gkamat pushed a commit that referenced this pull request Jun 10, 2024
(cherry picked from commit 411e304)

Signed-off-by: Govind Kamat <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
gkamat pushed a commit that referenced this pull request Jun 10, 2024
(cherry picked from commit 411e304)

Signed-off-by: Govind Kamat <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
gkamat pushed a commit that referenced this pull request Jun 10, 2024
(cherry picked from commit 411e304)

Signed-off-by: Govind Kamat <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@gkamat gkamat changed the title Add missing quotes to _big5_ Jinja template Add missing quotes to big5 Jinja template Jun 10, 2024
@gkamat gkamat deleted the big5-typo branch June 20, 2024 16:38
harshavamsi pushed a commit to harshavamsi/opensearch-benchmark-workloads that referenced this pull request Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1 backport 2 Backport to the "2" branch backport 3 Backport to the "3" branch backport 7 Backport to the "7" branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants