Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Code Intelligence Sprint 5 Tracking issue #17853

Closed
31 of 36 tasks
efritz opened this issue Jan 29, 2021 · 10 comments
Closed
31 of 36 tasks

Code Intelligence Sprint 5 Tracking issue #17853

efritz opened this issue Jan 29, 2021 · 10 comments
Assignees
@efritz @scalabilitysolved @olafurpg @macraig @garobrik @Strum355
Labels
team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) tracking
Milestone
Code Intelligence...

Comments

@efritz
Copy link
Contributor

efritz commented Jan 29, 2021 •
edited
Loading

Plan

This sprint covers 2021-02-01 through 2021-01-12.

Availability

If you have planned unavailability this iteration (e.g., vacation), you can note that here.

Eric: 10d
Noah: 10d
Garo: 6d
Owen: 8d
Olaf: 10d
Maria: 10d
TJ: 10d

Tracked issues

@Strum355

  • Cross-language API/IDL support (GraphQL, Thrift, Protobuf) (#981)
  • Failed LSIF uploads not reflected in uploads UI (#18188) 🐛

Completed

  • (🏁 18 days ago) Improve debuggability (#27)
  • (🏁 18 days ago) Emit stacktrace on segfault with instructions for core-dump only scenarios (#32) :shipit:
  • (🏁 13 days ago) introduce documentation to aid users in the case of a segfault (#43) :shipit:
  • (🏁 12 days ago) WIP: Documenting lsif-java ft. minor refactorings (#83) :shipit:
  • (🏁 12 days ago) cross-package type alias definitions have self-referencing semantic definition results (#130; PRs: #131) 🐛

@efritz

  • Remove long-running transactions from dbworker (#14920)
  • blog: Write about the code intelligence commit graph optimization effort (#17379)

Completed

  • (🏁 19 days ago) Search-based code intel hovers are broken when definition token and docstring are separated by multiple lines (#17925; PRs: #567)
  • (🏁 18 days ago) Add clarity to MinIO credential settings (#16412; PRs: #17965)
  • (🏁 17 days ago) codeintel: Ensure txn is used for bulk inserts (#18020) :shipit:
  • (🏁 12 days ago) cross-package type alias definitions have self-referencing semantic definition results (#130; PRs: #131) 🐛
  • (🏁 10 days ago) UI: Fix cache key for document highlights (#17997) :shipit:
  • (🏁 10 days ago) Fix pings for lsif WAUs (#17923; PRs: #17959)
  • (🏁 10 days ago) chore: Add correct size hints when making slices (#18211) :shipit:
  • (🏁 10 days ago) chore: Fix order of driver.Value declarations in dbutil (#18212) :shipit:
  • (🏁 10 days ago) chore: Distinguish stores in error messages (#18213) :shipit:
  • (🏁 10 days ago) chore: Update bloom filter package interface (#18223) :shipit:
  • (🏁 10 days ago) codeintel: Add GetDumpsByIDs to dbstore (#18214) :shipit:
  • (🏁 10 days ago) codeintel: Add BulkMonikerResults to lsifstore (#18217) :shipit:
  • (🏁 10 days ago) codeintel: Standardize limit/offset parameters in lsifstore (#18218) :shipit:
  • (🏁 10 days ago) codeintel: Add DefinitionDumps and ReferenceIDsAndFilters to dbstore (#18221) :shipit:
  • (🏁 10 days ago) codeintel: Add PagedReferences to lsifstore (#18215) :shipit:
  • (🏁 10 days ago) codeintel: Do not return nil scanner (#18226) :shipit:
  • (🏁 3 days ago) codeintel: Remove dead code (api package, dbstore, lsifstore) (#18231) :shipit:
  • (🏁 today) codeintel: Add a request-level data cache (#15943; PRs: #17964)
  • (🏁 today) codeintel: Batch lsif store requests (#15945; PRs: #17964)

@gbrik

  • Disconnect between hover for the same symbol (#39)

Completed

  • (🏁 21 days ago) make code intel alerts non dismissible (#566) :shipit:
  • (🏁 21 days ago) RFC 201: Do not auto-index things being manually indexed (#14572; PRs: #17893)
  • (🏁 19 days ago) codeintel: prepopulate index configuration field with auto-configurator output (#17489; PRs: #17736)
  • (🏁 18 days ago) write script to combine multiple LSIF dumps of the same repo locally before upload (#17960; PRs: #24)
  • (🏁 14 days ago) escape the project root we compare paths against (#42) :shipit:

@tjdevries

Completed

  • (🏁 18 days ago) ci: Use https:// rather than ssh for clone (#41) :shipit:
  • (🏁 12 days ago) cross-package type alias definitions have self-referencing semantic definition results (#130; PRs: #131) 🐛

Legend

  • 👩 Customer issue
  • 🐛 Bug
  • 🧶 Technical debt
  • 🎩 Quality of life
  • 🛠️ Roadmap
  • 🕵️ Spike
  • 🔒 Security issue
  • :shipit: Pull Request
@efritz efritz added team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) tracking labels Jan 29, 2021
@efritz efritz added this to the Code Intelligence Sprint 5 milestone Jan 29, 2021
@github-actions
Copy link
Contributor

Heads up @macraig - the "team/code-intelligence" label was applied to this issue.

@efritz efritz mentioned this issue Feb 1, 2021
Closed
40 tasks
@garobrik
Copy link
Contributor

garobrik commented Feb 2, 2021 •
edited
Loading

(going to start updating this as I go since it doesn't happen otherwise)

Week 1 Done

Week 2 Done

Week 2 Planned

  • Update docs based on issues found during HRT call
  • Investigate a weird range mismatch in precise hovers for lsif-clang (and other langs?) Disconnect between hover for the same symbol lsif-clang#39
  • Since I have some extra time, maybe write an RFC about some overdue code reorganization we've been putting off

@macraig
Copy link
Contributor

macraig commented Feb 2, 2021 •
edited
Loading

Last week:

  • RFC 293
  • Customer info aggregation: CE syncs, dashboard scraping, Sales/CE information surfacing project
  • Customer reach out and follow-ups

This week:

  • Planning prep
  • RFCs 294 and 307
  • Fill in alignment doc
  • Kick off design collab
  • Customer follow ups
  • OKR review and brainstorming

@olafurpg
Copy link
Member

olafurpg commented Feb 5, 2021

This week:

  • Reviewed protoc brainstorm by @Strum355
  • Investigated lsif-java and the Kythe Java indexer
  • Started working on a new Java compiler plugin based off lsif-java, but a clean rewrite in Java
  • Addressed several comments on RFC 307

@macraig
Copy link
Contributor

macraig commented Feb 8, 2021

Last week:

  • Code intel OKRs and strategy
  • Customers follow ups
  • RFCs 294 and 307
  • Alignment doc
  • Dashboard get back WAUs
  • Review cycle
  • Design collab draft doc

This week:

  • Q1 OKRs alignment and roadmap adjusting
  • Continue customer reach out and delivery follow up
  • Planning prep
  • Chorus AI digging
  • Draft mid-long term code intel roadmap

@efritz
Copy link
Contributor Author

efritz commented Feb 8, 2021

Last week:

  • Looked in to database size issues at customer
  • Looked in to performance issues at prospect customer (will continue investigating)

This week:

Going to continue working on backend performance optimizations.

@macraig
Copy link
Contributor

macraig commented Feb 12, 2021

Last week:

This week:

  • Customer followup
  • Merge updated goals/milestones
  • More Chorus AI digging

@efritz
Copy link
Contributor Author

efritz commented Feb 12, 2021

This week:

Investigated a few customer performance issues. Refactored the code intelligence resolvers in https://github.com/sourcegraph/sourcegraph/pull/17964 to make it obvious where we need to optimize. Made a bunch of issues with the code-intelligence-performance label.

Next week:

Will plan/prioritize the issues with the label above and start digging in to that for the next few iterations.

@olafurpg
Copy link
Member

Last week:

This week:

  • Gradle integration for lsif-java

@efritz
Copy link
Contributor Author

efritz commented Feb 15, 2021

Continued in https://github.com/sourcegraph/sourcegraph/issues/18290.

@efritz efritz closed this as completed Feb 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@efritz efritz

@scalabilitysolved scalabilitysolved

@olafurpg olafurpg

@macraig macraig

@garobrik garobrik

@Strum355 Strum355

Labels
team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) tracking
Projects
None yet
Milestone
Code Intelligence Sprint 5
Development

No branches or pull requests
6 participants
@efritz @scalabilitysolved @olafurpg @macraig @garobrik @Strum355