Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow searching for members #868

Closed
jelaplan opened this issue Jun 12, 2020 · 20 comments
Closed

Allow searching for members #868

jelaplan opened this issue Jun 12, 2020 · 20 comments
Assignees
Labels
enhancement New feature or request

Comments

@jelaplan
Copy link

Problem
Users can't search for members of a dataset. They can only search or data sets themselves.

Solution
Allow searching for the names of members as a more detailed thing to access.

@bearou
Copy link

bearou commented Aug 25, 2020

It is very important for out organisation, that some kind of search possibility or filter is provided, as our source code is placed in one (!) dataset per environment. Imagine scrolling a member list in Zowe/VS Code with more than 200K members...

@MikeBauerCA
Copy link
Contributor

You can filter the document trees in Visual Studio Code -> https://code.visualstudio.com/docs/getstarted/userinterface#_filtering-the-document-tree

Below is a quick GIF to see it in action. I'd be interested to see if it works with so many members.
VSCode_filter

@jellypuno
Copy link
Contributor

jellypuno commented Aug 25, 2020

you can use Ctrl + ALT + P to filter the members that you are looking for. This was implemented by PR #592

@jellypuno
Copy link
Contributor

I'd be interested to see if it works with so many members.

@MikeBauerCA Yes it works. This was tested before and results can be found in issue #327

@MikeBauerCA
Copy link
Contributor

@jellypuno after opening the PDS to see the 30k members, did you try to perform the local filter? Was it immediate?

@jellypuno
Copy link
Contributor

@MikeBauerCA We have an issue with the filtering 😆 . I opened issue #992 .

@MikeBauerCA
Copy link
Contributor

You can still filter the trees using the method I described above though. Not sure if you still have that large PDS to try it out on.

@bearou
Copy link

bearou commented Aug 26, 2020

Have used the "filter the document trees in Visual Studio Code" for the first time... Wonder why I havent seen this before !?
It seems like it filters within the VS Code "frontend", hence our problem is not solved as we have 200K+ in one single production source library... usually this is not an issue...

@bearou
Copy link

bearou commented Aug 26, 2020

In regards to Ctrl + ALT + P, a few of us in the office have tried it, and we all get "Enter pattern..."
Where do we learn what "pattern" should look like, since nothing happens, no matter what we type in ?

@MikeBauerCA
Copy link
Contributor

It appears we need to enhance the Zowe Node SDK to support the start and pattern query parameters of the REST endpoint (https://www.ibm.com/support/knowledgecenter/SSLTBW_2.4.0/com.ibm.zos.v2r4.izua700/IZUHPINFO_API_GetListDataSetMembers.htm) and then leverage this in both Zowe Explorer and Zowe CLI.

@bearou
Copy link

bearou commented Sep 2, 2020

It appears we need to enhance the Zowe Node SDK to support the start and pattern query parameters of the REST endpoint (https://www.ibm.com/support/knowledgecenter/SSLTBW_2.4.0/com.ibm.zos.v2r4.izua700/IZUHPINFO_API_GetListDataSetMembers.htm) and then leverage this in both Zowe Explorer and Zowe CLI.

Out of curiosity... what about setting the X-IBM-Max-Items ? It appears that we do not receive the total number of members in the dataset.

@martindamp
Copy link

It appears we need to enhance the Zowe Node SDK to support the start and pattern query parameters of the REST endpoint (https://www.ibm.com/support/knowledgecenter/SSLTBW_2.4.0/com.ibm.zos.v2r4.izua700/IZUHPINFO_API_GetListDataSetMembers.htm) and then leverage this in both Zowe Explorer and Zowe CLI.

Hi,
I did create an issue on support for this in zowe-cli and it has now been implemented zowe/zowe-cli#810 . The pull request still needs to be approved but i think it will be soon.

So now we only need it to be implemented in Zowe Explorer.

@JillieBeanSim JillieBeanSim self-assigned this Nov 11, 2020
@JillieBeanSim
Copy link
Contributor

JillieBeanSim commented Nov 12, 2020

My plan going forward on this is to first accept member name or member pattern in the initial search filter of a profile and no longer throw an error if this is entered like it does now
pds-member-filter-error

Following that we could add an additional approach being a search filter magnifying glass to data sets names or possibly a right click action on data set names to search for a member pattern within that data set.

@JillieBeanSim
Copy link
Contributor

JillieBeanSim commented Nov 17, 2020

@jelaplan here is my work in progress with allowing member search in the filter search bar, what do you think?
the search entered into the filter is zowebjs.test*.*(test*), zowebjs.testnds(test*)
pds-member-filter-working

If no member filter added to the search filter zowebjs.test*.*, zowebjs.testnds I get this list of members
image

@JillieBeanSim
Copy link
Contributor

We were discussing as a team about having a new folder icon with a filter created for data set members that have a filter member search attached to them.

@jelaplan
Copy link
Author

@crawr and @jellypuno I think it is a great start. I like your suggestion to pick a dataset and then filter the members to make it a two step process where the user can drill down - I think that it would be more obvious/apparent that filtering members can be done.

Following that we could add an additional approach being a search filter magnifying glass to data sets names or possibly a right click action on data set names to search for a member pattern within that data set.

I think the dataset(member) syntax is a standard convention in ISPF. But, it might not be all that obvious to the ZE user. We might want to change the quick pick instruction. Perhaps @BrandonJenkins14 or others could offer some suggestions. The current text is

+Create a new filter. Comma separate multiple entries (pattern 1, pattern 2, ...)

But we might try putting in an example. Someone with more MF exp probably can suggest something more accurate.

+Create a new filter. For example, hlq.aaa.bbb(member), hlq.ccc.ddd*

I was not familiar with @MikeBauerCA 's filter approach. I think @bearou said the same thing. I don't think it works that well. In the example, marble appears in the data set name and the member names and so a filter on mable does narrow it down. But, where the member name doesn't appear in the dataset, filtering ends up hiding everything.

WRT filtering with cmd+option+p (ctrl+alt+p), I also can't figure that out. I've tried typing a lot of combinations but can't get it to work.
image

Per @JillieBeanSim, a folder icon that indicates an active filter seems promising. It would give some feedback to the user that there is a filter in place. I could make that icon and share it with the team to see if people approve.

@crawr
Copy link
Contributor

crawr commented Feb 16, 2021

With the implemented feature in v1.12.0 regarding dataset member filtering, can we now close this issue? @jelaplan @JillieBeanSim

@JillieBeanSim
Copy link
Contributor

JillieBeanSim commented Feb 17, 2021

With the implemented feature in v1.12.0 regarding dataset member filtering, can we now close this issue? @jelaplan @JillieBeanSim

@crawr Did we ever decide if we wanted to also have a right-click action on a data set to filter the members within it? I think that is why it wasn't attached to the PR. If what we have is good or we could create a new issue to add the enhancement at a later time then I vote to close it

@jelaplan
Copy link
Author

jelaplan commented Mar 2, 2021

WRT right click filtering / drill down filtering on a data set, maybe you should create a new story for that. I wouldn't want that to hold things up. I still think it is often easier to narrow in on something step-by-step to make the syntax simpler, but what you've done is good so start with that.

@JillieBeanSim
Copy link
Contributor

New issue created for the right-click action on PDS for member filter search within the tree view #1241
Closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

7 participants