https://lakefs.io/ logo
Title
v

Vaibhav Kumar

02/23/2023, 3:15 PM
Hi Team I am planning to work on this issue Please tag me to this. Github vaibhavk1992 And want to understand more around the issue. Please help me with more details.
๐Ÿ‘€ 1
:jumping-lakefs: 3
๐Ÿ™Œ 1
i

Itai Admi

02/23/2023, 3:19 PM
Hey @Vaibhav Kumar ๐Ÿ‘‹ Weโ€™re always happy to accept contributions! Letโ€™s continue the discussion on the issue itself
v

Vaibhav Kumar

02/23/2023, 4:06 PM
done
๐Ÿ™ 1
@Itai Admi Have posted some question on the github. Kindly answer
๐Ÿ‘ 1
Do I have to create a new provider package in this issue(A new package all together) or I have to include it in lakfe-provider-package code?
i

Itai Admi

02/25/2023, 3:26 PM
I'm pretty sure it belongs to the lakeFS provider package. Does the Airflow docs suggest otherwise? I can check it again tomorrow
v

Vaibhav Kumar

02/25/2023, 4:49 PM
No ,it doesnt looks in the airflow side But just confused whether i need to create a new provider package or need to creates files within lakefs provider
@Itai Admi I am able to get a connection form with lakeFS type on airflow. Was this the only thing to be done in this issue?
image.png
i

Itai Admi

02/28/2023, 6:35 PM
That's amazing, great job! Please open a pr and add me as a reviewer
v

Vaibhav Kumar

02/28/2023, 6:40 PM
Will do it. Meanwhile please assign this issue on my github id.
๐Ÿ‘ 1
what could be the next issue to pick? This time I am looking for something in Lakefs repo. I have listed out some of them but not sure how much effort will be needed for each of them https://github.com/treeverse/lakeFS/issues/4908 https://github.com/treeverse/lakeFS/issues/2801 https://github.com/treeverse/lakeFS/issues/4535
i

Itai Admi

03/01/2023, 6:14 PM
I would suggest you filter by the
good-first-issue
label
v

Vaibhav Kumar

03/01/2023, 6:17 PM
yeah, Actually 2 and 3 are
good-first-issue
but I was curious as well if I can pick the first one related to terraform
Any update around the PR review?
i

Itai Admi

03/02/2023, 9:00 AM
I see that @Ariel Shaqed (Scolnicov) started reviewing, thank you!
Regarding the next items, the terraform provider issue is linked to our recent efforts around authorization & authentication. Since these 2 are key elements for any audit review, the team thought it would be best to not label it as
good-first-issue
. For the third item, I requested some product input, Iโ€™m unsure that we want to follow the suggested path described in the issue. The second issue would benefit the lakeFS project a lot. Our error messages in the HadoopFS can be improved a lot. Are you familiar with Hadoop/Spark or curious about learning more about it? If so, I think thatโ€™s the best option out of the 3 issues.
v

Vaibhav Kumar

03/02/2023, 9:18 AM
Yes, I am a data engineer by profession. I do understand Hadoop/Spark/Hive/Cloud. ๐Ÿ™‚
:60fps_parrot: 1
:jiggling-lakefs: 1
Hi @Ariel Shaqed (Scolnicov) /johnnyaug On the last PR raised I was trying to follow how can we test connection created. I was following this link. I am not sure how shall I use test_connection in my hook. Can you please share some example or direction how shall I use it my hook?
@Itai Admi @Ariel Shaqed (Scolnicov) I have re worked on the PR and tested all the things this time. it is working fine. Please verify it once.