I'm interested in taking a stab at this!

I'm interested in taking a stab at this!

Go ahead! LMK if you have any questions!

@TalZaccai , can you please update the assignees? This will help others easily find the unpicked items.
If this one has not been picked up yet, I'm interested.

If more than one person wants to attempt an open work item, please go ahead. If a PR comes in quicker and meets the requirements, then we would be happy to merge it.

Someone else can take this. I don't think I have time to tackle this in a timely manner right now.

@MojtabaTajik Let me know if you'd like to try and tackle this one!

@TalZaccai yes, pls assign it to me.

@TalZaccai yes, pls assign it to me.

Done! :)

@MojtabaTajik - any update on this?

@TalZaccai @badrishc

I just started working on it this week. Since it's my first contribution to Garnet, it's taking me more time to understand everything.

I've completed all the steps, but in step 7, there's a partial class called "SetObject" which contains a HashSet<byte[]> field named "set".
I've written the "SetIntersect" method there, allowing me to retrieve set names requested by the user (setsToIntersect variable). I also have the current set in the "set" field. However, I'm not sure how to retrieve other set members here and perform the intersection operation.

Am I proceeding correctly here?

        private void SetIntersect(byte* input, int length, ref SpanByteAndMemory output)
        {
            var _input = (ObjectInputHeader*)input;
            int numSets = _input->count;

            bool isMemory = false;
            MemoryHandle ptrHandle = default;
            
            // Parse the input to get the sets you want to intersect
            List<HashSet<byte[]>> setsToIntersect = new List<HashSet<byte[]>>();
            byte* inputCurrPtr = input + sizeof(ObjectInputHeader);
            for (int i = 0; i < numSets; i++)
            {
                if (!RespReadUtils.ReadByteArrayWithLengthHeader(out var setBytes, ref inputCurrPtr, input + length))
                    return; // Handle parsing error

                var tempSect = new HashSet<byte[]>() { setBytes };
                setsToIntersect.Add(tempSect);
            }

            // Perform the intersection
            var currentSet = set;
            
        }

@TalZaccai @badrishc

I just started working on it this week. Since it's my first contribution to Garnet, it's taking me more time to understand everything.

I've completed all the steps, but in step 7, there's a partial class called "SetObject" which contains a HashSet<byte[]> field named "set". I've written the "SetIntersect" method there, allowing me to retrieve set names requested by the user (setsToIntersect variable). I also have the current set in the "set" field. However, I'm not sure how to retrieve other set members here and perform the intersection operation.

Am I proceeding correctly here?

        private void SetIntersect(byte* input, int length, ref SpanByteAndMemory output)
        {
            var _input = (ObjectInputHeader*)input;
            int numSets = _input->count;

            bool isMemory = false;
            MemoryHandle ptrHandle = default;
            
            // Parse the input to get the sets you want to intersect
            List<HashSet<byte[]>> setsToIntersect = new List<HashSet<byte[]>>();
            byte* inputCurrPtr = input + sizeof(ObjectInputHeader);
            for (int i = 0; i < numSets; i++)
            {
                if (!RespReadUtils.ReadByteArrayWithLengthHeader(out var setBytes, ref inputCurrPtr, input + length))
                    return; // Handle parsing error

                var tempSect = new HashSet<byte[]>() { setBytes };
                setsToIntersect.Add(tempSect);
            }

            // Perform the intersection
            var currentSet = set;
            
        }

Hey @MojtabaTajik!

I understand your confusion, I think originally when I wrote these instructions they were meant for adding a single-object operation, so given your input I might update those for future work items :)

To your question, your parsing logic should go into SetCommands.cs and the main command logic would go into SetOps.cs, where you can obtain the appropriate locks on the relevant keys and you could call the StorageSession's GET and SET operations to read & write to & from the object store.

I would recommend that you review some similar commands that were implemented recently, such as LMOVE, SDIFF, SDIFFSTORE & SUNION for reference.

Thanks again for your question,
Don't hesitate to reach out again.

Tal.

@TalZaccai
I was working on this on my weekends. I just saw a PR from Lapkarol that will implement them so you can use them.

@TalZaccai I was working on this on my weekends. I just saw a PR from Lapkarol that will implement them so you can use them.

Sounds good, I'll have a look at that new PR. Didn't know that someone else was working on it...