Status Update
Comments
uc...@google.com <uc...@google.com>
ad...@google.com <ad...@google.com> #2
Hey Vovkab! That's a really great point. And something I recall thinking about, but it had slipped my mind.
Pier, is there a reason we have that top-level node? Do we expect there to be other nodes at the same level as "Databases"?
vo...@gmail.com <vo...@gmail.com> #3
Great, thank you for looking into this issue!
ps...@google.com <ps...@google.com> #4
No, I agree it's not very useful. There were talks of separating closed and open dbs in two groups though, I don't recall ever making a decision on that.
If we decide to separate open and closed dbs I will add a new node. If not, I can just remove "Databases" node.
cs...@google.com <cs...@google.com> #5
I believe also we had plans of showing other data sources in that panel besides databases, like shared preferences. That would change the panel name I think if we add other sources of data to inspect.
ad...@google.com <ad...@google.com> #6
Sounds like we should remove that node for now. I agree that we should look into renaming the inside panel and reconsidering top-level nodes when we support other types of structured data.
cs...@google.com <cs...@google.com> #7
Agreed, let's remove the extraneous "Databases" node in the tree.
ad...@google.com <ad...@google.com> #8
Great! Pier, is this something we can get in for 4.1?
ps...@google.com <ps...@google.com> #9
The fix for this issue will be available in 4.1 beta 2.
Description
Build: AI-201.7223.91.41.6507185, 202005182153,
AI-201.7223.91.41.6507185, JRE 1.8.0_242-release-1644-b3-6222593x64 JetBrains s.r.o, OS Linux(amd64) v5.4.0-31-generic, screens 2560x1440
AS: 4.1 Canary 10; Kotlin plugin: 1.3.61-release-Studio4.1-1; Android Gradle Plugin: 3.6.0; Gradle: 6.1.1; NDK: from local.properties: 21.0.6113669, latest from SDK: 21.0.6113669; LLDB: pinned revision 3.1 not found, latest from SDK: (package not found); CMake: from local.properties: (not specified), latest from SDK: (not found), from PATH: 3.16.3
This is not exactly a bug, more of UX improvement suggestion.
Currently Database Inspector shows "Databases" twice:
Until there is a plan to show databases from multiple process I think second group could be removed. It would allowed us to save some space and make it a bit cleaner.
Please see attached screenshot for more details.