You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "dependabot[bot] (via GitHub)" <gi...@apache.org> on 2023/08/28 04:27:12 UTC
[GitHub] [beam] dependabot[bot] opened a new pull request, #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
dependabot[bot] opened a new pull request, #28180:
URL: https://github.com/apache/beam/pull/28180
Bumps [github.com/proullon/ramsql](https://github.com/proullon/ramsql) from 0.0.1 to 0.1.2.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/proullon/ramsql/releases">github.com/proullon/ramsql's releases</a>.</em></p>
<blockquote>
<h2>v0.1.2</h2>
<h2>What's Changed</h2>
<ul>
<li>fix: AttributeSelector returns error containing unfound attribute name by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/90">proullon/ramsql#90</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/proullon/ramsql/compare/v0.1.1...v0.1.2">https://github.com/proullon/ramsql/compare/v0.1.1...v0.1.2</a></p>
<h2>Fix regression on CamelCase attribute query</h2>
<h2>What's Changed</h2>
<ul>
<li>fix(engine): Relation.Attribute() convert name argument to lower case by <a href="https://github.com/trakhimenok"><code>@trakhimenok</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/87">proullon/ramsql#87</a></li>
<li>fix: (close <a href="https://redirect.github.com/proullon/ramsql/issues/86">#86</a>) AttributeSelector uses lower case to match attributes by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/88">proullon/ramsql#88</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/trakhimenok"><code>@trakhimenok</code></a> made their first contribution in <a href="https://redirect.github.com/proullon/ramsql/pull/87">proullon/ramsql#87</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/proullon/ramsql/compare/v0.1.0...v0.1.1">https://github.com/proullon/ramsql/compare/v0.1.0...v0.1.1</a></p>
<h2>Transaction based rewrite</h2>
<h2>Context</h2>
<p>Introduction of agnostic in-memory store, leveraging GC-optimized storage to garantee low GC pauses and good GC behavior over time.</p>
<p>Complete rewrite improving performance, removing deadlocks and storage as Go types, allowing creation of objects table.</p>
<p>Ensure compatibility with gorm, a popular Go ORM.</p>
<p>Added implementation goal in README, in preparation for stable 1.0 release.</p>
<h2>What's Changed</h2>
<ul>
<li>Implement SCHEMA by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/76">proullon/ramsql#76</a></li>
<li>feat: handle Gorm query style by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/77">proullon/ramsql#77</a></li>
<li>add/scientific notation by <a href="https://github.com/Joematpal"><code>@Joematpal</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/78">proullon/ramsql#78</a></li>
<li>Rewrite engine with SQL-agnostic GC optimized memory store by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/80">proullon/ramsql#80</a></li>
<li>Fix driver transaction behavior by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/84">proullon/ramsql#84</a></li>
<li>fix: insert mixed case value attribute by <a href="https://github.com/proullon"><code>@proullon</code></a> in <a href="https://redirect.github.com/proullon/ramsql/pull/85">proullon/ramsql#85</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/proullon/ramsql/compare/v0.0.1...v0.1.0">https://github.com/proullon/ramsql/compare/v0.0.1...v0.1.0</a></p>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/proullon/ramsql/commit/8cb2915e14a11df1cacca11c2f5899f187276d47"><code>8cb2915</code></a> fix: AttributeSelector returns error containing unfound attribute name</li>
<li><a href="https://github.com/proullon/ramsql/commit/71d06ede384fa4d187e9b3683028bb697c9d6873"><code>71d06ed</code></a> fix: AttributeSelector uses lower case to match attributes</li>
<li><a href="https://github.com/proullon/ramsql/commit/5b0278f228f61573b2b883e7626acea5e29ab65b"><code>5b0278f</code></a> fix: add failing test for CamelCase query</li>
<li><a href="https://github.com/proullon/ramsql/commit/481f3e2bae694a3f9f9f68811bb4263f31b2e54d"><code>481f3e2</code></a> fix(engine): Relation.Attribute() convert name argument to lower case to be c...</li>
<li><a href="https://github.com/proullon/ramsql/commit/e12c9872c05dfd96f12daef316ce7398581defac"><code>e12c987</code></a> fix: insert mixed case value attribute</li>
<li><a href="https://github.com/proullon/ramsql/commit/a7e15a63022d55a5ba4a643e74b31e4af06e842e"><code>a7e15a6</code></a> fix: use driver.Conn as Tx</li>
<li><a href="https://github.com/proullon/ramsql/commit/d851e3f8ea466e7ba8aa02c7c152ab99a570903c"><code>d851e3f</code></a> feat: extend transaction testing</li>
<li><a href="https://github.com/proullon/ramsql/commit/6014112392236a6e825491e59f61063afc59164a"><code>6014112</code></a> chore: update travis go version</li>
<li><a href="https://github.com/proullon/ramsql/commit/dc4783e951ecef9c34718f703af1c5f89021acd2"><code>dc4783e</code></a> feat: add test for <a href="https://redirect.github.com/proullon/ramsql/issues/40">#40</a></li>
<li><a href="https://github.com/proullon/ramsql/commit/7e9cda5993a2a02f5898ef0c797de166fb377119"><code>7e9cda5</code></a> chore: update benchstat</li>
<li>Additional commits viewable in <a href="https://github.com/proullon/ramsql/compare/v0.0.1...v0.1.2">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] commented on pull request #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #28180:
URL: https://github.com/apache/beam/pull/28180#issuecomment-1694993702
## [Codecov](https://app.codecov.io/gh/apache/beam/pull/28180?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
> Merging [#28180](https://app.codecov.io/gh/apache/beam/pull/28180?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (bdc9785) into [master](https://app.codecov.io/gh/apache/beam/commit/e26735d69f0935c6f7ac14dd0fb0e58ff390392a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (e26735d) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## master #28180 +/- ##
==========================================
+ Coverage 72.29% 72.30% +0.01%
==========================================
Files 678 678
Lines 99848 99848
==========================================
+ Hits 72189 72200 +11
+ Misses 26084 26070 -14
- Partials 1575 1578 +3
```
| [Flag](https://app.codecov.io/gh/apache/beam/pull/28180/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [go](https://app.codecov.io/gh/apache/beam/pull/28180/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `53.65% <ø> (+0.03%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
[see 1 file with indirect coverage changes](https://app.codecov.io/gh/apache/beam/pull/28180/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] lostluck merged pull request #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
Posted by "lostluck (via GitHub)" <gi...@apache.org>.
lostluck merged PR #28180:
URL: https://github.com/apache/beam/pull/28180
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] github-actions[bot] commented on pull request #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #28180:
URL: https://github.com/apache/beam/pull/28180#issuecomment-1695015756
Assigning reviewers. If you would like to opt out of this review, comment `assign to next reviewer`:
R: @jrmccluskey for label go.
Available commands:
- `stop reviewer notifications` - opt out of the automated review tooling
- `remind me after tests pass` - tag the comment author after tests pass
- `waiting on author` - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
The PR bot will only process comments in the main thread (not review comments).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] jrmccluskey commented on pull request #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
Posted by "jrmccluskey (via GitHub)" <gi...@apache.org>.
jrmccluskey commented on PR #28180:
URL: https://github.com/apache/beam/pull/28180#issuecomment-1696120759
@lostluck we'll want to skip this one, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] lostluck commented on pull request #28180: Bump github.com/proullon/ramsql from 0.0.1 to 0.1.2 in /sdks
Posted by "lostluck (via GitHub)" <gi...@apache.org>.
lostluck commented on PR #28180:
URL: https://github.com/apache/beam/pull/28180#issuecomment-1696236949
Should be fine now. I changed the type from string to *slog.Source earlier:
https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/runners/prism/internal/worker/worker.go#L204
And that's the right move as a reaction to the issue I filed with Go: https://github.com/golang/go/issues/62294
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org