On Fri, Jul 16, 2021 at 10:42 PM Sergei Golubchik wrote:
> Hi, Rucha!
>
> Hi, Sergei!
Thanks for the review.
> On Jul 15, Rucha Deodhar wrote:
> > revision-id: a11756e4fba (mariadb-10.5.11-27-ga11756e4fba)
> > parent(s): f0f47cbca18
> > author: Rucha Deodhar
were still showing as
binary files, which was probably because of the diffs. So I removed the
existing files from my repo and pushed new files then cleaned up the
commits again and fixed the 80 character per line rule for new tests. My
github repo is up to date with latest changes.
Regards,
Rucha
My github repo is up
to date with the latest changes.
Here is the link for the same:
https://github.com/rucha174/server/commit/0d7eb4198cc0aadba42acb4960c17570270423f5
Regards,
Rucha Deodhar
___
Mailing list: https://launchpad.net/~maria-developers
Post to
Hello!
Thanks for the review. I'll start working on it and keep you updated with
my progress.
Regards,
Rucha
On Wed, Aug 7, 2019 at 7:51 PM Vicențiu Ciorbaru
wrote:
> Hi Rucha!
>
> This is going to be a very thorough review looking at coding style. I'll
> nitpick on every line of code where I
rning val1='C'.
I have updated the result file for insert_returning.
I also fixed line endings for tests and added more comments to make our
implementation more understandable. I github repo is up to date with the
latest changes.
Regards,
Rucha Deodhar.
__
used, I
wanted to check if these functions can be used)
I have made related changes in the documentation and my github repo is up
to date with latest changes.
Regards,
Rucha Deodhar
___
Mailing list: https://launchpad.net/~maria-developers
Post to
NING a list was declared
inside the scope of "if" and using it outside the scope was illegal. So I
moved it to the scope outside of if and made the grammar return:
&list if select_lex::item_list is not empty else NULL.
My github repo is up to date with latest cha
according to the diff file and also fixed
REPLACE...RETURNING and have pushed the latest changes on my repo.
I also ran the tests related to INSERT...RETURNING. They are passing.
Also, I am willing to work on fixing some of the hacks once GSoC project
implementation is perfect.
Regards,
Rucha
Hello!
The review is indeed very helpful. I'll start working on the review today
and keep you updated about the progress.
Regards,
Rucha Deodhar
On Sun, Jul 21, 2019, 12:05 AM Vicențiu Ciorbaru
wrote:
> Hi Rucha!
>
> I've spent quite a bit of time looking at the im
nsert_update opt_select_expressions
{
something like
if ($3)
Lex->returning_list= $3;
Lex->pop_select(); //main select
if (Lex->check_main_unit_semantics())
MYSQL_YYABORT;
}
;
Regards,
Rucha Deodhar
<
al2) VALUES(4,'d') RETURNING (SELECT id1 FROM t1 where
id1=1);
Because it was returning only one row and column.
Regards,
Rucha Deodhar
On Sun, Jul 7, 2019 at 7:16 PM Vicențiu Ciorbaru
wrote:
> Hi Rucha!
>
>
> Here is the promised review. I have run the main test suite local
gards,
Rucha Deodhar
<https://www.avast.com/en-in/recommend?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=default3&tag=ad9cce02-ddec-4842-be2f-33f375f8223d>
I’m
protected online with Avast Free Antivirus. Get it here — it’s free foreve
Hello!
Thanks for the review. It will indeed help me write better code and test
cases.
To begin with, I' ll first get familiar with git bisect. I'll keep you
updated about my progress.
Regards,
Rucha Deodhar
On Sun, Jul 7, 2019, 7:16 PM Vicențiu Ciorbaru wrote:
> Hi Rucha!
>
is too big a change for the grammar and I am trying to
find alternate solution.
I also changed the variable names and made them more descriptive.
Regards,
Rucha Deodhar
___
Mailing list: https://launchpad.net/~maria-developers
Post to : maria
it by looking how select_insert class
uses lex->field_list and do the same for returning_list.
Here is the link to code which returns the listed fields for select table:
https://github.com/rucha174/server/commit/6b18dd3cfc76d41871357c5b2006e04d1d508c1d
R
mit/218d81b86d941a874cfac9123d7aeb141e01f110
Regards,
Rucha Deodhar
___
Mailing list: https://launchpad.net/~maria-developers
Post to : maria-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~maria-developers
More help : https://help.launchpad.net/ListHelp
Report for week 2:
Hello!
This week I ran the test case for parser written in last week and made
changes in the parser according to the suggestions given, tried making
insert...select..returning work.
I analysed how RETURNING in delete is implemented, how insert is
implemented, and how the data c
Report for week 1:
Hello!
I made test cases and extended the parser for:
INSERT...RETURNING * and
INSERT...RETURNING
It is working fine for INSERT...ON DUPLICATE KEY UPDATE, INSERT...IGNORE,
INSERT...SET, and simple insert statements.
As I have never coded parser before, it was throwing errors d
Hello,
I am Rucha Deodhar. I am a GSoC student and will implement
INSERT...RETURNING.
In the first week I plan to extend the parser. Approximately the next 2
months, I plan to make changes the in the codes in insert.cc and other
files related to the project, and remaining time for testing. As
Hello.
My name is Rucha Deodhar. I would like to contribute to mariaDB during GSoC
2019. More information can be found in my draft of proposal on the attached
google doc.
Any feedback would be appreciated.
Thanks
Proposal for “ADD RETURNING TO INSERT” for GSoC’19
<https://docs.google.
20 matches
Mail list logo