profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/spotify/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

spotify/annoy 8374

Approximate Nearest Neighbors in C++/Python optimized for memory usage and loading/saving to disk

spotify/chartify 2883

Python library that makes it easy for data scientists to create charts.

spotify/apollo 1610

Java libraries for writing composable microservices

spotify/apps-tutorial 622

A Spotify App that contains working examples of the use of Spotify Apps API

spotify/android-sdk 275

Spotify SDK for Android

spotify/big-data-rosetta-code 256

Code snippets for solving common big data problems in various platforms. Inspired by Rosetta Code

spotify/cassandra-reaper 235

Software to run automated repairs of cassandra

spotify/async-google-pubsub-client 151

[SUNSET] Async Google Pubsub Client

spotify/bazel-tools 134

Tools for dealing with very large Bazel-managed repositories

spotify/annoy-java 120

Approximate nearest neighbors in Java

PullRequestReviewEvent

PR opened spotify/ts-interface-loader

Bump ssri from 6.0.1 to 6.0.2

Bumps ssri from 6.0.1 to 6.0.2. <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/npm/ssri/blob/v6.0.2/CHANGELOG.md">ssri's changelog</a>.</em></p> <blockquote> <h2><a href="https://github.com/zkat/ssri/compare/v6.0.1...v6.0.2">6.0.2</a> (2021-04-07)</h2> <h3>Bug Fixes</h3> <ul> <li>backport regex change from 8.0.1 (<a href="https://github.com/zkat/ssri/commit/b30dfdb">b30dfdb</a>), closes <a href="https://github-redirect.dependabot.com/zkat/ssri/issues/19">#19</a></li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/npm/ssri/commit/b7c8c7c61db89aeb9fbf7596c0ef17071bc216ef"><code>b7c8c7c</code></a> chore(release): 6.0.2</li> <li><a href="https://github.com/npm/ssri/commit/b30dfdb00bb94ddc49a25a85a18fb27afafdfbb1"><code>b30dfdb</code></a> fix: backport regex change from 8.0.1</li> <li>See full diff in <a href="https://github.com/npm/ssri/compare/v6.0.1...v6.0.2">compare view</a></li> </ul> </details> <details> <summary>Maintainer changes</summary> <p>This version was pushed to npm by <a href="https://www.npmjs.com/~nlf">nlf</a>, a new releaser for ssri since your current version.</p> </details> <br />

Dependabot compatibility score

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.


<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 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)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+6 -6

0 comment

1 changed file

pr created time in 13 minutes

create barnchspotify/ts-interface-loader

branch : dependabot/npm_and_yarn/ssri-6.0.2

created branch time in 13 minutes

PR opened spotify/scio

Use the same hash fn in hashPartition and hashPartitionByKey

Just noticed while reviewing the code that the hash functions used for partitioning the lhs and the rhs can have slightly different behaviors in transforms like sparseIntersectByKey. Not sure if there is a practical implication of this.

+13 -10

0 comment

3 changed files

pr created time in 14 minutes

push eventspotify/scio

Neville Li

commit sha b97b87878d675394486cb20ac40df521a111b6d9

fix sparse transforms split on the wrong side (#3770) regression from https://github.com/spotify/scio/commit/a824db3bf0#diff-e957bccad978e20a8db993e00bf89c894f82521d980de57684402176351fd5b6L492-R435

view details

push time in 31 minutes

delete branch spotify/scio

delete branch : neville/bf-split

delete time in 31 minutes

PR merged spotify/scio

fix sparse transforms split on the wrong side

regression from https://github.com/spotify/scio/commit/a824db3bf0#diff-e957bccad978e20a8db993e00bf89c894f82521d980de57684402176351fd5b6L492-R435

I don't think it affect correctness. By using the potentially much larger LHS keys by mistake, we're saturating the BFs so that they'll return a lot of false positives, but they'll be filtered out in the exact join of "hot" sides. The only problem is it's not keeping enough "chill" keys from shuffling (BF return false, i.e. key not in RHS).

+1 -1

3 comments

1 changed file

nevillelyh

pr closed time in 31 minutes

pull request commentspotify/scio

fix sparse transforms split on the wrong side

@anish749 it was my copy/pasta error. Test didn't catch it since the result is always correct and data size was too small to reveal any issue.

nevillelyh

comment created time in 31 minutes

PullRequestReviewEvent
PullRequestReviewEvent

startedspotify/confidence

started time in 3 hours

pull request commentspotify/magnolify

Update munit-scalacheck to 0.7.25

Codecov Report

Merging #329 (b70664c) into main (09bdde3) will not change coverage. The diff coverage is n/a.

:exclamation: Current head b70664c differs from pull request most recent head a2b843e. Consider uploading reports for the commit a2b843e to get more accurate results Impacted file tree graph

@@           Coverage Diff           @@
##             main     #329   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files          44       44           
  Lines        1466     1466           
  Branches       71       71           
=======================================
  Hits         1416     1416           
  Misses         50       50           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 09bdde3...a2b843e. Read the comment docs.

scala-steward

comment created time in 3 hours

PR opened spotify/magnolify

Update munit-scalacheck to 0.7.25

Updates org.scalameta:munit-scalacheck from 0.7.23 to 0.7.25. GitHub Release Notes - Version Diff

I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.

If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.

Configure Scala Steward for your repository with a .scala-steward.conf file.

Have a fantastic day writing Scala!

<details> <summary>Ignore future updates</summary>

Add this to your .scala-steward.conf file to ignore future updates of this dependency:

updates.ignore = [ { groupId = "org.scalameta", artifactId = "munit-scalacheck" } ]

</details>

labels: test-library-update, semver-patch

+1 -1

0 comment

1 changed file

pr created time in 4 hours

startedspotify/flyte-flink-plugin

started time in 4 hours

startedspotify/flink-on-k8s-operator

started time in 4 hours

startedspotify/luigi

started time in 4 hours

startedspotify/web-api-auth-examples

started time in 4 hours

startedspotify/luigi

started time in 5 hours

push eventspotify/scio

Neville Li

commit sha 3d85013f06b4178d50e345c05b477cbb25a9ff39

update docs: Algebird => Guava BloomFilter

view details

push time in 5 hours

startedspotify/annoy

started time in 5 hours

Pull request review commentspotify/apollo

Make a HEAD /foo requests emit metrics for HEAD:/foo, not GET:/foo

   @Override   public EndpointRunnableFactory apply(EndpointRunnableFactory delegate) {     return (request, requestContext, endpoint) -> {-      final String endpointName = endpoint.info().getName();+      final String endpointName;+      // Normally request.request().method() and endpoint.info().getRequestMethod() are equal+      // but since apollo lets GET endpoints also serve HEAD requests, they are different+      // when the request is a HEAD request. The hack below makes sure we distinguish metrics+      // for GET and HEAD requests.+      // We could always take the else branch but I'm guessing that the comparison+      // is cheaper than a needless concatenation.+      if (request.request().method().equals(endpoint.info().getRequestMethod())) {+        endpointName = endpoint.info().getName();+      } else {+        // but because GET endpoints serve HEAD requests info().getName() is+        // wrong does not distinguish HEAD and GET requests+        endpointName = request.request().method() + ":" + endpoint.info().getUri();

Looks like this could work. But it would be more elegant if proper endpoint infos were set up for the implicit HEAD endpoints. This is a bit of a hack, isn't it?

mikaelgoldmann

comment created time in 5 hours

PullRequestReviewEvent

Pull request review commentspotify/semantic-metrics

Document Distributions

 The default histogram implementation uses a random sampling algorithm with expon This was authored by Johan Buratti.   +## Distribution+An aggregatable data structure, T-digest, that allows for aggregating percentiles. To get accurate percentiles, you need to setup good bucket boundaries.

Thanks for sharing that @ao2017 I used it to re-write the first part. Should be good for another review now

lmuhlha

comment created time in 5 hours

PullRequestReviewEvent

push eventspotify/semantic-metrics

Lauren Muhlhauser

commit sha 349c64cb31954c3f69e122f6804bab5f7ec375bc

Document Distributions

view details

push time in 5 hours

push eventspotify/semantic-metrics

Lauren Muhlhauser

commit sha 6380815501d21ebf75c01006cc19a70ac42529d4

Document Distributions

view details

push time in 5 hours

push eventspotify/semantic-metrics

Lauren Muhlhauser

commit sha 858cfda151ca71fab17e79071eadb2c425340a9d

Document Distributions

view details

push time in 5 hours

push eventspotify/semantic-metrics

Lauren Muhlhauser

commit sha e1c33ae002c3bcb29d8a6e45369ac6326798f442

Update histogram discription (#107)

view details

Lauren Muhlhauser

commit sha 2cf06cb1cdc5027dad01d540be4b61ef6cba9097

Document Distributions

view details

push time in 5 hours

pull request commentspotify/scio

Update case-app, case-app-annotations, ... to 2.0.6

@jto did you say we need to deprecate case-app due to shapeless/macro usage? Is there a ticket?

scala-steward

comment created time in 5 hours

pull request commentspotify/scio

fix sparse transforms split on the wrong side

@anish749 PTAL since you're probably most familiar with this part.

nevillelyh

comment created time in 6 hours