Skip to content

[go-borges] Use go-borges to access repositories#888

Merged
ajnavarro merged 16 commits into
src-d:masterfrom
jfontan:go-borges-rebase
Jun 20, 2019
Merged

[go-borges] Use go-borges to access repositories#888
ajnavarro merged 16 commits into
src-d:masterfrom
jfontan:go-borges-rebase

Conversation

@jfontan

@jfontan jfontan commented Jun 19, 2019

Copy link
Copy Markdown
Contributor

Note: These are the changes from branch go-borges-integration rebased with the latest version in master plus fixes to new tests.

  • Now gitbase uses go-borges to access repositories
    • The type of files in each directory has to be specified (#867)
    • Supports new rooted repository format and separates references and objects from each repo (New borges design borges#389)
  • Changed cli to be able to specify different formats (#866)

go-borges still does not support non rooted siva files (the ones generated by old borges). We are working on adding compatibility but all the changes for gitbase are done (even cli to support this). It may be better to release a final version of 0.21.0 and start a 0.22.0-beta with these changes.

  • I updated the documentation explaining the new behavior if any.
  • I updated CHANGELOG.md file adding the new feature or bug fix.
  • I updated go-mysql-server using make upgrade command if applicable.
  • I added or updated examples if applicable.
  • I checked that changes on schema are reflected into the documentation, if applicable.

@jfontan
jfontan requested review from a team June 19, 2019 17:28
Comment thread common_test.go Outdated
Comment thread common_test.go Outdated
Comment thread common_test.go Outdated
Comment thread repositories_test.go Outdated
Comment thread repository_pool_test.go Outdated
Comment thread repository_pool_test.go Outdated
Comment thread squash_iterator_test.go Outdated
@erizocosmico

Copy link
Copy Markdown
Contributor

Other than GolangCI stuff, if CI passes, LGTM

@jfontan
jfontan force-pushed the go-borges-rebase branch from bdefd6a to 93bde04 Compare June 20, 2019 08:29
jfontan added 16 commits June 20, 2019 10:45
Checksum:

Now repositories are not returned in an specific order. Checksum is
calculated per repository and ordered before calculating the final one.

Signed-off-by: Javi Fontan <jfontan@gmail.com>
Global format can be specified with --format, --bare, --bucket and
--rooted. To mix directories the format can be specified as a file URL:

    -d file:///path/to/library?format=siva&bucket=4&rooted=false

Also:

* Updated builder container to go 1.12
* Added SIVA env to docker init.sh to configure gitbase to use siva
library

Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Fixed documentation, typos and erroneous messages. Also library URL
parameters are converted to lowercase before parsing.

Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Also fix some repositories left open.

Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
@jfontan
jfontan force-pushed the go-borges-rebase branch from 93bde04 to 044094c Compare June 20, 2019 08:51
@jfontan

jfontan commented Jun 20, 2019

Copy link
Copy Markdown
Contributor Author

Rebased with latest changes in master and fixed linting errors.

@ajnavarro

Copy link
Copy Markdown
Contributor

ping @kuba-- @juanjux for a review

@ajnavarro
ajnavarro merged commit 13ed1b4 into src-d:master Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants